Skip to content

Builder, slice 2: GamGUI's curated commands, changes through ChangeCore - #140

Merged
goetchstone merged 2 commits into
mainfrom
builder/curated-commands
Oct 28, 2026
Merged

goetchstone merged 2 commits into
mainfrom
builder/curated-commands

Conversation

@goetchstone

Copy link
Copy Markdown
Owner

GamGUI's curated Builder commands with typed fields: reads run as the Builder's others do; a change is previewed through ChangeCore (BuilderChanges), confirmed with a click, and run once, exactly as held.

  • 24 of GamGUI's 26, held to GamGUI's fields and argv by Tests/Fixtures/catalog_curated.json (generated from a pinned GamGUI clone). Signature stays on Signatures; the delete waits for Off3.
  • Checks GamGUI's Builder lacks: addresses must be one plain address (isOneAddress); suspend and the random reset refuse the connected admin by the account GAM resolves, at preview and again before the run; the Users delegate check; a warning before title or department is cleared. "Headers + body" waits for a read audit.
  • Accessibility: every field and status is labelled in words; Run / Preview… on ⌘R; a destructive Run is never the default button.
  • New screenshots: builder-change, builder-confirm, narrow-builder-change.

Reviewed: gam-command-reviewer (one medium finding, fixed: uid:/<…> and alias bypasses of the admin refusal), /code-review, /security-review (no findings). Details in docs/domains/catalog-builder.md.

Tests: swift test green (957), app builds with no warnings. Live checks B2–B4 not run yet.

🤖 Generated with Claude Code

goetchstone and others added 2 commits October 28, 2026 11:29
24 of GamGUI's 26 curated commands with typed fields, held to GamGUI's
by catalog_curated.json. A change is previewed (BuilderChanges), confirmed
with a click and run once; suspend and the random password reset refuse
the connected admin by the account GAM resolves. Details in
docs/domains/catalog-builder.md.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…nge and confirm added

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@goetchstone
goetchstone enabled auto-merge October 28, 2026 16:38
@goetchstone
goetchstone merged commit 3394130 into main Oct 28, 2026
7 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