Skip to content

chore(bootstrap): pin 7ee3d5cc6416 and re-record digests - #89

Merged
bdelanghe merged 1 commit into
mainfrom
claude/front-desk-not-working-6993hp
Aug 1, 2026
Merged

chore(bootstrap): pin 7ee3d5cc6416 and re-record digests#89
bdelanghe merged 1 commit into
mainfrom
claude/front-desk-not-working-6993hp

Conversation

@bdelanghe

Copy link
Copy Markdown
Contributor

The bump #88 needs. By hand again, because the pin job still can't push its branch — #87, the App installation lacks contents: write.

SUM_session_start_dispatch_mjs moves; the other two are unchanged, as expected — #88 touched only the dispatcher.

Generated with node .claude/gen-bootstrap-pin.mjs 7ee3d5cc6416ecf9a5c37bce3b66fae62ac2599d, not hand-edited. Verified with GITHUB_EVENT_NAME=push, the setting that asserts FRESHNESS — i.e. under the condition that is currently red on main, not just the PR-mode one.

This is the second hand-bump today

Worth stating plainly rather than letting it look like routine noise: the pin goes stale only when a fetched file changes, so this isn't churn — it's the designed hand-off running with its automated half broken. Every PR touching .claude/ will need a follow-up like this until #87 is fixed, and if one is forgotten, main stays red and sessions without .github attached are served stale files from the fallback path. That is the failure bootstrap-pin.test.mjs was written to catch, and it catches it correctly; nothing is acting on the catch.


Generated by Claude Code

By hand again, for #88, because the `pin` job still cannot push its bump branch
(#87 — the App installation lacks `contents: write`).

Second hand-bump in one afternoon. The pin only goes stale when a fetched file
changes, so this is not noise: it is the designed hand-off running with its
automated half broken, and every PR touching `.claude/` will need this until #87
is fixed.
@bdelanghe
bdelanghe requested a review from a team as a code owner August 1, 2026 15:27
@bdelanghe
bdelanghe merged commit 2b5fba4 into main Aug 1, 2026
9 checks passed
@bdelanghe
bdelanghe deleted the claude/front-desk-not-working-6993hp branch August 1, 2026 15:28
@bounded-systems-front-desk bounded-systems-front-desk Bot moved this from Todo to Done in Front Desk Aug 1, 2026
bdelanghe added a commit that referenced this pull request Aug 3, 2026
Two changes, one cause (#87).

1. Point at `front-desk-pin`, not `front-desk`. The fan-in entry carries no
   `contents`, so it could never push the bump branch, and it cannot be given
   any: it is deliberately unpinned and `contents` is privileged, so the broker
   would refuse the entry outright and take every other consumer with it. The
   separate pinned entry is bounded-systems/infra#172.

2. Do not use the token when the mint step FAILED. `require: contents,
   pull_requests` (#93) worked exactly as designed on the merge of #97 — it
   reported `contents(granted: absent)` and named both places the gap could
   live. Then `continue-on-error: true` swallowed the verdict, the job used the
   token anyway, and died on the very push the assertion had just said would
   fail. An assertion whose verdict nothing consumes is decoration.

The fallback is not a downgrade: github.token holds contents:write here and
pushed this branch fine before the broker was wired in (#79). So on a scope gap
the branch now LANDS with the correct pin and only opening the PR is lost — a
click, versus the full hand-regenerate it costs today (#79, #86, #89, #98).

A third annotation separates "broker reachable, scopes insufficient" from
"broker unreachable", since the two are fixed in different systems.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants