Claude/confirm doc context loading r0 pa5

[Censseo]

No description provided.

[Copilot]

Pull request overview

This PR expands Spec Kit’s template system to provide richer spec/project context artifacts, adds multiple “skills” reference packs, introduces an MCP project-server template, and updates command templates to include semantic anchors and improved workflow guidance.

Changes:

• Add new spec/plan/idea/feature/context templates and multiple new skill reference bundles.
• Add MCP “project-server” TypeScript template (process mgmt, API testing, browser automation scaffolding) and update docs to reference new workflows.
• Update automation scripts (agent-context tech consolidation; bug listing) and adjust packaging/release scripts and Python package include rules.

Reviewed changes

Copilot reviewed 69 out of 69 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
templates/specs-context-template.md	New template for consolidated project state context for specs/ work.
templates/spec-template.md	Adds constitution-compliance guidance + new mandatory/conditional sections.
templates/skills/tech-debt/references/refactoring-strategies.md	New tech-debt reference content.
templates/skills/tech-debt/references/debt-patterns.md	New tech-debt patterns catalog.
templates/skills/tech-debt/SKILL.md	New “tech-debt” skill definition + triggers.
templates/skills/microservices-patterns/references/resilience-patterns.md	New microservices resilience reference content.
templates/skills/microservices-patterns/references/data-patterns.md	New microservices data patterns reference content.
templates/skills/microservices-patterns/SKILL.md	New “microservices-patterns” skill definition + triggers.
templates/skills/code-review/references/security-review.md	New security code review checklist reference.
templates/skills/code-review/references/review-checklist.md	New comprehensive review checklist reference.
templates/skills/code-review/SKILL.md	New “code-review” skill definition + triggers.
templates/skills/architecture-patterns/references/implementations.md	New architecture implementations reference examples.
templates/skills/architecture-patterns/references/ddd-patterns.md	New DDD patterns reference content.
templates/skills/architecture-patterns/SKILL.md	New “architecture-patterns” skill definition + triggers.
templates/skills/architecture-decision-records/references/templates.md	New ADR templates reference.
templates/skills/architecture-decision-records/references/examples.md	New ADR examples reference.
templates/skills/architecture-decision-records/SKILL.md	New “architecture-decision-records” skill definition + triggers.
templates/plan-template.md	Adds progress tracking checklist section.
templates/module-claude-template.md	New module guidelines template (generated by /speckit.learn).
templates/mcp/project-server/tsconfig.json	New TS config for MCP server template.
templates/mcp/project-server/src/tools/process.ts	New process/log/health/docker tools for MCP server template.
templates/mcp/project-server/src/tools/api.ts	New API client/testing utilities for MCP server template.
templates/mcp/project-server/src/config.ts	New MCP server config template with placeholders/examples.
templates/mcp/project-server/package.json	New MCP server package template (deps/scripts).
templates/mcp/project-server/README.md	Documentation for using the MCP server template.
templates/idea-template.md	New structured idea template (vision → scope → risks → tech hints).
templates/feature-template.md	New decomposed feature template (scope/use-cases/deps/tech hints).
templates/commands/tasks.md	Strengthens tasks generation: idea alignment + reuse traceability.
templates/commands/setup.md	Adds setup orchestrator command template.
templates/commands/setup-skills.md	Adds stack detection → skill generation command template.
templates/commands/setup-mcp.md	Adds MCP server generation command template.
templates/commands/setup-docs.md	Adds docs initialization command template.
templates/commands/setup-constitution.md	Updates constitution setup guidance + propagation expectations.
templates/commands/semantic-anchors.md	Adds a central semantic anchors reference.
templates/commands/merge.md	Adds merge+docs consolidation command template.
templates/commands/implement.md	Adds semantic anchors + agent delegation + reduced upfront context load.
templates/commands/clarify.md	Adds semantic anchors + explicit elicitation framing.
templates/commands/checklist.md	Adds generate/review modes + constitution/registry auto-check scaffolding.
templates/commands/analyze.md	Adds semantic anchors + clearer analysis framing.
scripts/powershell/update-agent-context.ps1	Consolidates/deduplicates tech stacks in agent context updates.
scripts/powershell/list-bugs.ps1	New script to list unresolved validation bugs (PS).
scripts/bash/update-agent-context.sh	Consolidates/deduplicates tech stacks in agent context updates (bash).
scripts/bash/list-bugs.sh	New script to list unresolved validation bugs (bash).
pyproject.toml	Bumps version and ensures templates/scripts/memory are packaged.
memory/constitution.md	Reworks constitution template into explicit spec/dev/business sections.
memory/architecture-registry.md	Adds architecture registry template for cross-feature consistency.
docs/toc.yml	Adds workflows doc to TOC.
docs/quickstart.md	Adds guidance for /speckit.change and links to workflow guide.
docs/index.md	Links to workflow guide.
README.md	Updates command taxonomy + adds “full vs quick” workflow overview.
.github/workflows/scripts/create-release-packages.sh	Removes unsupported agents from packaging matrix.
.github/workflows/scripts/create-release-packages.ps1	Removes unsupported agents from packaging matrix (PS).
.github/workflows/scripts/create-github-release.sh	Removes unsupported agent artifacts from release creation.

Comments suppressed due to low confidence (1)

templates/commands/setup-constitution.md:50

• This instruction refers to RATIFICATION_DATE / LAST_AMENDED_DATE, but the updated memory/constitution.md template no longer contains those placeholders (it uses [DATE] for both Ratified/Last Amended). Either update the constitution template to use explicit [RATIFICATION_DATE]/[LAST_AMENDED_DATE] placeholders, or update this command text so it matches the actual template fields.

---

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

[Copilot]

Because this script uses set -e, the arithmetic increments like ((total++)) / ((open++)) can cause the script to exit early: in bash, ((expr)) returns exit status 1 when the expression evaluates to 0 (e.g., the first total++ evaluates to 0). Use pre-increment (((++total)), ((++open)), etc.) or a form that always returns 0 to avoid unexpected termination.

Suggested change

	open) ((open++)) ;;
	in_progress) ((in_progress++)) ;;
	resolved) ((resolved++)) ;;
	wont_fix) ((wont_fix++)) ;;
	esac
	case "$severity" in
	critical) ((critical++)) ;;
	high) ((high++)) ;;
	medium) ((medium++)) ;;
	low) ((low++)) ;;
	open) ((++open)) ;;
	in_progress) ((++in_progress)) ;;
	resolved) ((++resolved)) ;;
	wont_fix) ((++wont_fix)) ;;
	esac
	case "$severity" in
	critical) ((++critical)) ;;
	high) ((++high)) ;;
	medium) ((++medium)) ;;
	low) ((++low)) ;;

[Copilot]

new RegExp(filter, "i") will throw if the user passes an invalid regex (e.g., [), which would crash the tool call. Consider guarding with try/catch and returning a friendly error, or treating the filter as a literal substring (or escaping it) unless the API explicitly promises regex semantics.

[Copilot]

This template references .specify/memory/constitution.md, but other templates in this PR (and the repo’s memory/constitution.md) reference /memory/constitution.md. Please standardize the path to the constitution so generated specs don’t point at a non-existent location depending on install layout.

[Copilot]

There’s an internal contradiction here: step says “If file exists, append to existing file”, but the next bullet says each /speckit.checklist run creates a NEW file (never overwrites existing checklists). Please make the behavior unambiguous (either always create a new uniquely named file per run, or explicitly append/update an existing domain checklist).

Suggested change

	- If file exists, append to existing file
	- Number items sequentially starting from CHK001
	- Each `/speckit.checklist` run creates a NEW file (never overwrites existing checklists)
	- If file exists, append to the existing file instead of overwriting it
	- Number items sequentially starting from CHK001
	- Each `/speckit.checklist` run REUSES the domain file if it exists, appending new items and never deleting or overwriting existing checklist content

[Copilot]

The validation step suggests running node .mcp/project-server/dist/index.js --test, but the generated MCP server entrypoint doesn’t appear to implement a --test flag. This will lead to a confusing failure during setup. Either add --test handling to the template server, or update the docs to use an existing validation path (e.g., npm run inspect or a documented tool invocation).

[Copilot]

Unused variable frame.

[Copilot]

This assignment to 'arg_format' is unnecessary as it is redefined before this value is used.

Suggested change

arg_format = "$ARGUMENTS"

[Copilot]

This assignment to 'file_ext' is unnecessary as it is redefined before this value is used.

Suggested change

file_ext = "md"