Skip to content

docs: spec-kit pilot for state-sync component#1309

Draft
jeremyeder wants to merge 1 commit intoambient-code:mainfrom
jeremyeder:spec-kit/state-sync-pilot
Draft

docs: spec-kit pilot for state-sync component#1309
jeremyeder wants to merge 1 commit intoambient-code:mainfrom
jeremyeder:spec-kit/state-sync-pilot

Conversation

@jeremyeder
Copy link
Copy Markdown
Contributor

Summary

  • Adds behavioral specification for the state-sync sidecar (components/runners/state-sync/spec/spec.md)
  • Adds development context file (.claude/context/state-sync-development.md) following existing pattern
  • Adds BOOKMARKS.md entries for progressive disclosure

Pilots spec-driven development on the smallest runner component before rolling out to ambient-runner.

Test plan

  • Verify BOOKMARKS.md links resolve to real files
  • Review spec invariants against hydrate.sh and sync.sh source
  • Review development context format against backend-development.md

🤖 Generated with Claude Code

Behavioral specification and development context for the state-sync
sidecar, piloting spec-driven development before rolling out to
ambient-runner.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 14, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: f41924ac-982f-4441-84f5-2a81968371a3

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
✨ Simplify code
  • Create PR with simplified code

Comment @coderabbitai help to get the list of available commands and usage tips.

@markturansky
Copy link
Copy Markdown
Contributor

@jeremyeder this is an excellent extraction of the runner spec. It has all the config/params for the runner and backend APIs required by it.

I think it would be useful to also specify the runtime of the runner itself. It has multiple threads internally, acting as source and sink. There are bridges to other models besides Claude. We'll add gRPC to the spec as well (when we merge back from alpha).

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.

2 participants