Skip to content

feat: POST /api/assign (Asgard handshake, no LLM) - #9

Merged
jordanpartridge merged 2 commits into
mainfrom
feat/api-assign
Aug 19, 2026
Merged

feat: POST /api/assign (Asgard handshake, no LLM)#9
jordanpartridge merged 2 commits into
mainfrom
feat/api-assign

Conversation

@jordanpartridge

Copy link
Copy Markdown
Contributor

Door for Asgard #133 (`kit_assign`). Overnight builder #1108 will POST here.

```http
POST /api/assign
Authorization: Bearer KIT_PEER_TOKEN
{ "issue": "https://github.com/the-shit/bikes-v2/issues/12\", "chair": "kit", "brief": "rider stills" }
```

  • ≤200ms. No `Runner::say`. No `on it.`
  • Board row `lifecycle=queued`, catalog id if the brief names rider/hero-ebike/ranch-7620
  • Static hallway line; skipped if hallway is still the Jordan DM
  • `php artisan test --compact` 28 passed

Not this PR: `DispatchKitWork` hops, `GitHubOpenIssue`.

jordan added 2 commits August 18, 2026 22:54
Asgard #133 / Kit handshake. Bearer KIT_PEER_TOKEN. Board queued.
Hallway ack only — never the Jordan DM. No DispatchKitWork yet.

@lexi-chief-of-staff lexi-chief-of-staff Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ship it.

  • Token check, upsert, catalog board_id, and hallway skip all match spec.
  • Test coverage includes auth, required fields, happy path, and DM skip.
  • One nit: line 79 double-assign is dead code; harmless but clean it up before merge.

No blocking issues. 28 tests green.

@lexi-chief-of-staff lexi-chief-of-staff Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ship it.

  • Endpoint wires the Asgard POST exactly as specified, with token auth, board upsert, and conditional hallway post.
  • boardId correctly maps to catalog ids on brief/issue match; skips DM hallway per config.
  • Tests cover auth failure, missing issue, happy path with mock, and DM skip.
  • No dispatch or LLM paths touched; 28 tests pass.

@jordanpartridge
jordanpartridge merged commit 3429447 into main Aug 19, 2026
2 checks passed
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.

1 participant