Refine opsx archive sync assessment

[TabishB]

Updates opsx archive instructions to assess delta spec sync state agentically and report a combined summary before prompting. Adds clearer prompt options and sync status wording.

Summary by CodeRabbit

• Refactor
  • Reorganized sync-assessment flow: added an explicit "Assess delta spec sync state" step with per-delta-spec evaluation and a combined summary before prompting.
  • Updated sync status wording across skill prompts (e.g., "Sync skipped" replaces prior labels) and simplified multi-step reporting into single assessment-and-prompt flows.
• New Features
  • When delta specs exist, users now get clear sync/ archive options based on a detailed comparison; when none exist, prompts are skipped.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Warning

Rate limit exceeded

@TabishB has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 9 minutes and 28 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 67e9157 and 5c3be9f.

📒 Files selected for processing (1)

• src/core/templates/skill-templates.ts

📝 Walkthrough

Walkthrough

Replaces the prior "quick sync check" with an explicit "Assess delta spec sync state" flow in skill templates: per-delta-spec comparisons against main specs, classification of required actions, a combined summary, and updated prompt/archival messaging; prompts are skipped when no delta specs exist.

Changes

Cohort / File(s)	Summary
Delta-spec sync assessment refactor src/core/templates/skill-templates.ts	Unified sync flow: removed scattered "quick sync check" references and introduced an "Assess delta spec sync state" step that parses delta specs, compares each to main specs, classifies actions (ADD/MODIFY/REMOVE/RENAME/REPAIR), produces a combined summary, and updates prompts and archive messaging; skips sync prompt when no delta specs exist.

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant SkillTemplate as Skill Template
  participant DeltaSpecs as Delta Specs
  participant MainSpecs as Main Specs

  User->>SkillTemplate: invoke archive/sync action
  SkillTemplate->>DeltaSpecs: check for delta specs
  alt delta specs exist
    SkillTemplate->>MainSpecs: load main specs
    SkillTemplate->>DeltaSpecs: iterate & compare per-delta-spec
    SkillTemplate->>SkillTemplate: classify actions (ADD/MODIFY/REMOVE/RENAME/REPAIR)
    SkillTemplate->>User: present combined summary + prompt (sync/archive options)
    User->>SkillTemplate: choose action
  else no delta specs
    SkillTemplate->>User: proceed without sync prompt (archive flow)
  end

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~22 minutes

Possibly related PRs

• feat: add /opsx:sync command for syncing delta specs to main specs #450 — Introduces related sync/opsx:sync skill changes touching the same templates and sync logic.
• feat: add /opsx:archive command for archiving completed changes #451 — Edits archive/sync templates and messaging overlapping with this refactor.
• feat: add smart sync check to /opsx:archive command #452 — Adds the earlier "quick sync-check" flow that this PR replaces with the detailed assessment.

Poem

🐰 I nibble through deltas, one by one,
Comparing specs beneath the sun.
I tally adds, renames, and repairs,
Then offer choices with careful airs —
Hop, sync, archive — my work is done! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The PR title "Refine opsx archive sync assessment" is directly related to the main change in the changeset, which involves updating archive sync assessment flow and delta spec synchronization logic across skill templates.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

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 and usage tips.}

[vibe-kanban-cloud]

Review Complete

Your review story is ready!

View Story

Comment !reviewfast on this PR to re-generate the story.