Skip to content

docs: document Articles IV, V & VI as project-customizable in spec-driven.md#2467

Open
Copilot wants to merge 5 commits intomainfrom
copilot/docs-spec-driven-update-articles-iv-v-vi
Open

docs: document Articles IV, V & VI as project-customizable in spec-driven.md#2467
Copilot wants to merge 5 commits intomainfrom
copilot/docs-spec-driven-update-articles-iv-v-vi

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 6, 2026

spec-driven.md claimed "nine articles" but only documented six — Articles IV, V, and VI were silently missing between Article III and Articles VII & VIII.

Changes

  • spec-driven.md — adds #### Articles IV, V & VI: Project-Customizable Standards between Article III and Articles VII & VIII:
    • Identifies each article's domain: IV → integration concerns, V → observability/operational concerns, VI → versioning/lifecycle concerns
    • Explicitly marks these as project-defined (unlike I–III and VII–IX, which are framework-fixed)
    • References the [PRINCIPLE_4_NAME] / [PRINCIPLE_5_NAME] placeholders in templates/constitution-template.md as the customization point
    • Notes that /speckit.analyze already validates dynamically against whatever a project defines — no Spec Kit changes required to extend these articles

Copilot AI self-assigned this May 6, 2026
Copilot AI review requested due to automatic review settings May 6, 2026 13:51
Copilot AI review requested due to automatic review settings May 6, 2026 13:51
Copilot AI requested review from Copilot and removed request for Copilot May 6, 2026 13:54
Copilot AI changed the title [WIP] Document Articles IV, V, VI as project-customizable in spec-driven.md docs: document Articles IV, V & VI as project-customizable in spec-driven.md May 6, 2026
Copilot AI requested a review from mnriem May 6, 2026 13:55
Copilot finished work on behalf of mnriem May 6, 2026 13:55
@mnriem mnriem marked this pull request as ready for review May 6, 2026 16:38
@mnriem mnriem requested review from Copilot and removed request for Copilot May 6, 2026 16:38
@mnriem mnriem requested review from Copilot and removed request for Copilot May 6, 2026 19:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 3

Comment thread spec-driven.md Outdated
Comment thread spec-driven.md Outdated
Comment thread spec-driven.md Outdated
@mnriem
Copy link
Copy Markdown
Collaborator

mnriem commented May 6, 2026

@copilot apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 6, 2026

@copilot apply changes based on the comments in this thread

Applied all three review comments in d4ec0b7:

  1. Removed the "fixed in the Spec Kit framework" framing — the section now describes IV–VI as the typical customization focus areas while noting that /speckit.constitution lets teams define fewer or more principles than the template.
  2. Collapsed Article V and VI into a single bullet that correctly reflects the template grouping them under one [PRINCIPLE_5_NAME] / [PRINCIPLE_5_DESCRIPTION] slot by default, with a note that teams can expand to separate articles via /speckit.constitution.
  3. Reworded the /speckit.analyze sentence to say it validates all spec/plan artifacts against every principle in the project constitution, rather than implying article-number-specific logic.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 2

Comment thread spec-driven.md Outdated

- **Article IV** addresses integration concerns—such as integration testing requirements, contract testing across service boundaries, or cross-component communication standards. The constitution template's `[PRINCIPLE_4_NAME]` / `[PRINCIPLE_4_DESCRIPTION]` slot is dedicated to this article, with an example hint of *Integration Testing*.

- **Articles V & VI** address operational and lifecycle concerns—such as observability, structured logging, versioning schemes, or breaking-change policies. By default, the template groups these under a single `[PRINCIPLE_5_NAME]` / `[PRINCIPLE_5_DESCRIPTION]` slot (example hint: *V. Observability, VI. Versioning & Breaking Changes*). Teams that need separate articles can expand to additional principle sections using `/speckit.constitution`.
Comment thread spec-driven.md Outdated

- **Articles V & VI** address operational and lifecycle concerns—such as observability, structured logging, versioning schemes, or breaking-change policies. By default, the template groups these under a single `[PRINCIPLE_5_NAME]` / `[PRINCIPLE_5_DESCRIPTION]` slot (example hint: *V. Observability, VI. Versioning & Breaking Changes*). Teams that need separate articles can expand to additional principle sections using `/speckit.constitution`.

To define these articles, populate the corresponding principle sections in `.specify/memory/constitution.md` (generated from `templates/constitution-template.md`). The `/speckit.analyze` command validates all spec and plan artifacts against every principle in your project constitution, so any rules you define here are automatically applied during analysis.
@mnriem
Copy link
Copy Markdown
Collaborator

mnriem commented May 6, 2026

@copilot apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 6, 2026

@copilot apply changes based on the comments in this thread

Applied both comments in b9c7a0c:

  1. Updated the [PRINCIPLE_5_*] example hint to include "VII. Simplicity" so it exactly matches the template: V. Observability, VI. Versioning & Breaking Changes, VII. Simplicity.
  2. Updated the path reference to mention both locations: .specify/templates/constitution-template.md (in an initialized project) and templates/constitution-template.md (in the Spec Kit repo).

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 2

Comment thread spec-driven.md Outdated

- **Article IV** addresses integration concerns—such as integration testing requirements, contract testing across service boundaries, or cross-component communication standards. The constitution template's `[PRINCIPLE_4_NAME]` / `[PRINCIPLE_4_DESCRIPTION]` slot is dedicated to this article, with an example hint of *Integration Testing*.

- **Articles V & VI** address operational and lifecycle concerns—such as observability, structured logging, versioning schemes, or breaking-change policies. By default, the template groups these (along with a Simplicity article) under a single `[PRINCIPLE_5_NAME]` / `[PRINCIPLE_5_DESCRIPTION]` slot (example hint: *V. Observability, VI. Versioning & Breaking Changes, VII. Simplicity*). Teams that need separate articles can expand to additional principle sections using `/speckit.constitution`.
Comment thread spec-driven.md Outdated

- **Articles V & VI** address operational and lifecycle concerns—such as observability, structured logging, versioning schemes, or breaking-change policies. By default, the template groups these (along with a Simplicity article) under a single `[PRINCIPLE_5_NAME]` / `[PRINCIPLE_5_DESCRIPTION]` slot (example hint: *V. Observability, VI. Versioning & Breaking Changes, VII. Simplicity*). Teams that need separate articles can expand to additional principle sections using `/speckit.constitution`.

To define these articles, populate the corresponding principle sections in `.specify/memory/constitution.md` (initialized from `.specify/templates/constitution-template.md` in your project, sourced from `templates/constitution-template.md` in the Spec Kit repo). The `/speckit.analyze` command validates all spec and plan artifacts against every principle in your project constitution, so any rules you define here are automatically applied during analysis.
@mnriem
Copy link
Copy Markdown
Collaborator

mnriem commented May 6, 2026

@copilot apply changes based on the comments in this thread

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 6, 2026

@copilot apply changes based on the comments in this thread

Applied both comments in 88e9823:

  1. Removed "along with a Simplicity article" and added a clarifying note that the [PRINCIPLE_5_*] example hint's label text is a suggestion only — teams can name and scope that slot however fits their project. This makes clear there's no conflict with the framework-provided "Articles VII & VIII" section below.
  2. Standardized path notation to the short form memory/constitution.md (matching the existing convention at line 276), with the full .specify/memory/constitution.md path noted parenthetically for context.

Copilot finished work on behalf of mnriem May 6, 2026 19:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

docs: Document Articles IV, V, VI as project-customizable in spec-driven.md

3 participants