Skip to content

docs(osapi): break the agent key store into tasks - #135

Merged
retr0h merged 1 commit into
mainfrom
feat/agent-key-store-tasks
Sep 18, 2026
Merged

retr0h merged 1 commit into
mainfrom
feat/agent-key-store-tasks

Conversation

@retr0h

@retr0h retr0h commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

Phase 2 output for components/osapi/specs/002-agent-key-store: 41 tasks
organised by the spec's three user stories.

  • Setup / Foundational (T001-T013) — the store, its three distinct failure
    causes, the cache invalidated on write and remove, and acceptance as the only
    writer. Blocks everything else.
  • US1 (T014-T019, P1) — verify a job response against the stored key.
    Closes GHSA-3jh4's deferred half.
  • US2 (T020-T028, P1) — sign the registration, and make target resolution
    ignore unverified ones and resolve a contested hostname deterministically.
    Closes GHSA-j73r.
  • US3 (T029-T034, P2) — rotation with a bounded grace period, and removal.
  • Polish (T035-T041) — fleet view, the docs, coverage back to 99.9%, the
    full gate, and adding the fix to both advisories.

US1 and US2 are independent: both read the store, neither writes it, and they
touch different files.

Every file path in the task list was checked against the osapi tree rather than
taken from the plan. That caught two corrections — the canonical-serialisation
helper is a new internal/job/registration.go while the struct itself lives in
internal/job/types.go, and the key-held fields belong on AgentInfo, not
AgentRegistration, since AgentInfo is what ListAgents returns.

Gate: just test green (mdformat, just-fmt, skill-lint).

🤖 Generated with Claude Code

https://claude.ai/code/session_01FuKUsHFG1EqZXamffh9M2c

Organise the per-agent public key store into 41 tasks across the spec's
three user stories, with every cited path verified against the osapi tree.

US1 closes GHSA-3jh4's deferred response-verification half, US2 closes
GHSA-j73r, and US3 adds rotation and removal so enforcement does not
become a reason to switch verification back off. US1 and US2 are
independent and either can ship first.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FuKUsHFG1EqZXamffh9M2c
@github-actions

Copy link
Copy Markdown

Thank you for contributing to this project! 😊🕹️

@retr0h
retr0h merged commit b37e924 into main Sep 18, 2026
6 checks passed
@retr0h
retr0h deleted the feat/agent-key-store-tasks branch September 18, 2026 04:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant