Skip to content

feat(kanban): deliberate single-sub worker pin --pin-sub <reason> (t_957ca870) - #1318

Merged
ang-fleet-lander[bot] merged 1 commit into
mainfrom
daedalus/t_957ca870-pin-sub
Sep 27, 2026
Merged

ang-fleet-lander[bot] merged 1 commit into
mainfrom
daedalus/t_957ca870-pin-sub

Conversation

@ang-fleet-workers

@ang-fleet-workers ang-fleet-workers Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

feat(kanban): deliberate single-sub worker pin --pin-sub (t_957ca870)

Restores pinning a card/lane to one Claude sub (claude-bpx-N/apx-N),
which #1116 removed with a blanket refusal, behind an explicit reason:

  • create / set-model (+batch) / lane-model set take --pin-sub REASON and
    --pin-sub-fallback; stored in tasks.pin_sub_reason/pin_sub_fallback and
    lane_model_overrides.pin_sub_reason (migrated); 'sub pin:' audit comment.
  • Without the flag single-sub routes stay refused (text now names the pool
    and the flag instead of 'workers never pin'). Pre-rename aliases are
    refused even with the flag. Admission reads usage-registry.json (row
    enabled; apx refused during burn-in); unreadable registry fails closed.
    Sub 0 is pinnable as a last resort (pool-side reservation protects it).
  • Dispatcher: card pin route source=pin (lane: pin(lane-override(..)));
    a capped/cooling/circuit-open pinned sub WAITS (deferred, pin_fallback=
    wait) instead of taking the profile ladder; --pin-sub-fallback rides the
    family pool when admissible. Flagship + load gates unchanged.
  • show/list print [PIN claude-bpx-N: reason]; new 'kanban pins [--json]
    [--stale-hours N]' lists live pins (exit 1 on stale = daily lint).
  • Any unpinned route write / edit --model clears the pin.

Verified: test_kanban_sub_pin_optin.py 28 + test_kanban_pinned_sub_refusal.py
25 pass; neighbour kanban route/lane/pool/effort suites 194 pass. 8 mutants
(predicate ignores flag, registry skipped, pin hold removed, source=pin
removed, stale pin kept, fallback ignored, badge dropped, apx burn-in check
dropped) each fail the new suite. 4 flagship tests fail identically on
clean fork/main in a worker session (UNHOMED session guard) - inherited.

Card: t_957ca870


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

…957ca870)

Restores pinning a card/lane to one Claude sub (claude-bpx-N/apx-N),
which #1116 removed with a blanket refusal, behind an explicit reason:

- create / set-model (+batch) / lane-model set take --pin-sub REASON and
  --pin-sub-fallback; stored in tasks.pin_sub_reason/pin_sub_fallback and
  lane_model_overrides.pin_sub_reason (migrated); 'sub pin:' audit comment.
- Without the flag single-sub routes stay refused (text now names the pool
  and the flag instead of 'workers never pin'). Pre-rename aliases are
  refused even with the flag. Admission reads usage-registry.json (row
  enabled; apx refused during burn-in); unreadable registry fails closed.
  Sub 0 is pinnable as a last resort (pool-side reservation protects it).
- Dispatcher: card pin route source=pin (lane: pin(lane-override(..)));
  a capped/cooling/circuit-open pinned sub WAITS (deferred, pin_fallback=
  wait) instead of taking the profile ladder; --pin-sub-fallback rides the
  family pool when admissible. Flagship + load gates unchanged.
- show/list print [PIN claude-bpx-N: reason]; new 'kanban pins [--json]
  [--stale-hours N]' lists live pins (exit 1 on stale = daily lint).
- Any unpinned route write / edit --model clears the pin.

Verified: test_kanban_sub_pin_optin.py 28 + test_kanban_pinned_sub_refusal.py
25 pass; neighbour kanban route/lane/pool/effort suites 194 pass. 8 mutants
(predicate ignores flag, registry skipped, pin hold removed, source=pin
removed, stale pin kept, fallback ignored, badge dropped, apx burn-in check
dropped) each fail the new suite. 4 flagship tests fail identically on
clean fork/main in a worker session (UNHOMED session guard) - inherited.

Card: t_957ca870
@ang-fleet-lander

Copy link
Copy Markdown

🤖 merged-by: apollo · lane: kanban-merge-pass · gate: ADVISORY (FleetReview not green for 498b65f): fleetreview-advisory-20260927-standing.md · why: t_957ca870: FORK FEATURE: deliberate single-sub worker pin (--pin-sub ) — restore th; Argus off card review (Ace 13:08), CI green

@ang-fleet-lander
ang-fleet-lander Bot added this pull request to the merge queue Sep 27, 2026
Merged via the queue into main with commit d0d387a Sep 27, 2026
42 checks passed
@ang-fleet-lander
ang-fleet-lander Bot deleted the daedalus/t_957ca870-pin-sub branch September 27, 2026 16:51
@ang-fleet-ci-actuators ang-fleet-ci-actuators Bot added the fleetreview:post-merge Ask FleetReview to review this MERGED pull (merge commit vs first parent) label Sep 27, 2026
@ang-prism

ang-prism Bot commented Sep 27, 2026

Copy link
Copy Markdown

FleetReview

FleetReview's daily member-call budget is spent (1200/1200 for 2026-09-27 UTC); review skipped.


FleetReview · reviewKind: skipped-budget

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fleetreview:post-merge Ask FleetReview to review this MERGED pull (merge commit vs first parent)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants