fix(templates): use store-aware root for main specs in sync/archive

[clay-good]

Status: Ready. Closes #1358.

What was wrong
OpenSpec 1.5.0 added --store <id> to target a registered store, but two workflow instructions still assumed the main specs always live in the current repository:

• sync-specs read and created openspec/specs/<capability>/spec.md
• archive-change compared delta specs against that same hardcoded path before offering to sync

With a store selected, the change and its main specs belong to the store, while the repo may also have its own openspec/. So a sync could update the repo's specs instead of the store's, the store could stay out of date, and archive could report "already synced" based on the wrong location.

How it was fixed
The CLI already returns a store-aware planningHome.root in openspec status --json. The instructions now derive every main-spec path from it:

<planningHome.root>/openspec/specs/<capability>/spec.md

• sync-specs: both the main-spec read path and the new-main-spec creation path derive from planningHome.root (added a note in the "Resolve change context" step so the agent uses it).
• archive-change: the sync-assessment comparison uses that same derived path.
• Dropped the "repo-local" label on CLI-returned delta paths, since they may belong to a selected store.

Repo-local behavior (no store) is unchanged: planningHome.root is the repo root there.

Proof it works
Generated-content and function-payload golden hashes in test/core/templates/skill-templates-parity.test.ts were regenerated and pass, confirming the intended (and only the intended) instruction text changed. Full suite green except the 17 pre-existing environment-only zsh-installer failures (oh-my-zsh), which are unrelated.

✓ test/core/templates/skill-templates-parity.test.ts (6 tests)
✓ test/vocabulary-sweep.test.ts (2 tests)
Tests  1858 passed | 17 failed (zsh-installer, env-only)

Notes
Scope is the source workflow templates only. The committed skills/ distribution (PR #1357) isn't on main yet; when it lands, npm run generate:skills will pick up this text automatically.

🤖 Generated with Claude Code

Summary by CodeRabbit

• Bug Fixes
  • Improved specification synchronization and archiving workflows to correctly locate main specifications in the configured planning store.
  • Updated workflow guidance to support specifications stored outside the repository’s default path.
• Tests
  • Updated template integrity checks to reflect the revised workflow instructions.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 0d7208bf-3fc3-4b57-8e21-fe02537f4e30

📥 Commits

Reviewing files that changed from the base of the PR and between 0a99f41 and 5c89c61.

📒 Files selected for processing (3)

• src/core/templates/workflows/archive-change.ts
• src/core/templates/workflows/sync-specs.ts
• test/core/templates/skill-templates-parity.test.ts

---

📝 Walkthrough

Walkthrough

Updated generated sync and archive workflow instructions to resolve main-spec paths through planningHome.root, including store-scoped delta handling and new-spec creation. Updated parity test hashes for the affected templates and generated skills.

Changes

Store-aware workflow paths

Layer / File(s)	Summary
Sync and archive path guidance src/core/templates/workflows/sync-specs.ts, src/core/templates/workflows/archive-change.ts, test/core/templates/skill-templates-parity.test.ts	Sync and archive templates now use <planningHome.root>/openspec/specs/<capability>/spec.md for main-spec reads, comparisons, and creation; parity hashes were updated accordingly.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• Fission-AI/OpenSpec#1089: Updates the same workflow templates for planning-home-aware delta and main spec paths.
• Fission-AI/OpenSpec#632: Changes archive workflow sync invocation in the same template area.
• Fission-AI/OpenSpec#452: Implements related delta-versus-main synchronization checks.

Suggested reviewers: tabishb, alfred-openspec

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly summarizes the core change: making sync/archive templates use a store-aware root.
Linked Issues check	✅ Passed	The PR implements the linked requirements by using planningHome.root for main-spec paths, updating archive comparisons, and removing repo-local wording.
Out of Scope Changes check	✅ Passed	The test hash updates appear to support the template changes and no unrelated code changes are introduced.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}