Skip to content

docs: clarify project-defined constitution articles#2994

Merged
mnriem merged 1 commit into
github:mainfrom
YNan-varamor:docs/2466-project-defined-articles
Jun 23, 2026
Merged

docs: clarify project-defined constitution articles#2994
mnriem merged 1 commit into
github:mainfrom
YNan-varamor:docs/2466-project-defined-articles

Conversation

@YNan-varamor

Copy link
Copy Markdown
Contributor

Summary

Clarifies that Articles IV, V, and VI in the SDD constitution are project-defined governance slots rather than fixed Spec Kit rules.

Why

Issue #2466 notes that spec-driven.md describes Articles I, II, III, VII, VIII, and IX, but does not explain how Articles IV, V, and VI should be understood. The added section documents that these articles are intentionally customized by each project's constitution while preserving the nine-article structure.

Validation

  • Ran git diff --check
  • Confirmed /speckit.analyze is an existing command in the repository docs/templates

AI assistance disclosure

This PR was prepared with assistance from OpenAI Codex. The change was reviewed locally and kept to a focused documentation update.

Copilot AI review requested due to automatic review settings June 16, 2026 13:07

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds clarification to the spec-driven documentation describing how Articles IV–VI are intended to be project-defined via a constitution template, and how /speckit.analyze enforces those customized principles.

Changes:

  • Documented that Articles IV–VI are placeholders filled by each project’s constitution rather than fixed by Spec Kit
  • Clarified examples of what governance topics these articles may cover (security, integration testing, observability, versioning)
  • Stated that /speckit.analyze evaluates the project’s concrete constitution so these articles participate in compliance checks

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

Comment thread spec-driven.md

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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: 0 new

@YNan-varamor YNan-varamor marked this pull request as ready for review June 23, 2026 04:43
@YNan-varamor YNan-varamor requested a review from mnriem as a code owner June 23, 2026 04:43
@mnriem mnriem requested a review from Copilot June 23, 2026 12:04

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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: 1

Comment thread spec-driven.md

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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: 1

Comment thread spec-driven.md
@mnriem mnriem merged commit 3cfc81f into github:main Jun 23, 2026
11 checks passed
@mnriem

mnriem commented Jun 23, 2026

Copy link
Copy Markdown
Collaborator

Thank you!

@YNan-varamor YNan-varamor deleted the docs/2466-project-defined-articles branch June 24, 2026 05:53
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.

3 participants