fix: preserve constitution.md during reinitialization

[jin-joung]

Summary

Fixes #1541 by moving the constitution template from memory/ to templates/ to prevent overwrites when spec-kit is reinitialized with a different AI agent.

Changes

• Move constitution template: memory/constitution.md → templates/constitution-template.md
• Update CLI initialization: Added ensure_constitution_from_template() function that copies the template to memory only on first initialization
• Update constitution command: Updated paths to use .specify/ prefix for clarity
• Preserve existing constitution: When reinitializing, the CLI now checks if .specify/memory/constitution.md exists and preserves it

Behavior

The CLI now handles constitution initialization as follows:

• First initialization: Copies constitution-template.md from templates to memory
• Reinitialization: Preserves existing constitution.md in memory (no overwrite)

This allows users to customize their constitution without losing changes when adding support for additional AI agents or reinitializing the project.

Testing

• ✅ Constitution file successfully moved from memory/ to templates/
• ✅ Python CLI code compiles without syntax errors
• ✅ Package creation script includes correct structure

Note: Full end-to-end testing on Linux/CI will verify the complete workflow, as there's a pre-existing macOS-specific issue with the package creation script's use of cp --parents.

AI Assistance

This PR was implemented with assistance from Claude Sonnet 4.5 (Claude Code).

[Copilot]

Pull request overview

This pull request fixes issue #1541 by relocating the constitution template from memory/ to templates/ to prevent user customizations from being overwritten during project reinitialization. The solution introduces a conditional copy mechanism that only initializes the constitution from the template on first setup.

Changes:

• New constitution template file added in templates/ directory with placeholder tokens for project-specific values
• CLI initialization enhanced with ensure_constitution_from_template() function that preserves existing constitutions
• Command documentation updated to reflect the new .specify/memory/constitution.md path convention

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated 1 comment.

File	Description
templates/constitution-template.md	New template file with placeholder tokens for project name, principles, sections, and governance rules
templates/commands/constitution.md	Updated all constitution path references to use .specify/ prefix for consistency
src/specify_cli/init.py	Added constitution setup function with preservation logic and integrated it into the init workflow

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The tracker.add() calls are redundant here since the "constitution" step is already added to the tracker before this function is called (line 1146). The function should just call tracker.skip() directly without the preceding tracker.add() call, similar to how other tracker methods are used throughout the codebase (e.g., lines 1181, 1183). This same issue exists on lines 960, 969, and 975.