Skip to content

feat(routing): global defaults and guided setup wizard - #46

Open
GrillerGeek wants to merge 2 commits into
mainfrom
GlobalRouting
Open

GrillerGeek wants to merge 2 commits into
mainfrom
GlobalRouting

Conversation

@GrillerGeek

@GrillerGeek GrillerGeek commented Sep 25, 2026 •

Copy link
Copy Markdown
Owner

Guildhall currently requires routing setup in each project. This adds global per-host defaults with persistent approval, plus a conversational guildhall-routing-setup wizard so users can configure routing without editing JSON.

  • Resolve session selection, then a complete project policy or opt-out, then the current host's global policy under XDG_CONFIG_HOME or ~/.config/guildhall/.
  • Persist approval separately, bound to source, scope, policy hash, host configuration and reviewed evidence. Preserve expiry, revocation, task-specific summary consent and qualification checks.
  • Guide users through scope, objective, supported candidates and mode; recommend global shadow setup for new users. Preview before saving, with save-off/cancel paths and returning-user change/diagnose/disable flows.
  • Allow explicit setup inspection/activation of a global entry behind an unchanged project override. Worker routing still uses ordinary precedence and honors opt-out.
  • Ship independently installable quest and setup skills from shared canonical routing resources; bump all manifests to 0.16.0.

Migration: project policies replace global defaults in full. Deleting a project policy restores inheritance; use a project opt-out or session off to disable routing. Setup does not activate automatically during installation.

Validation

  • 154 tests passed on Python 3.12.2 and 3.14.5.
  • Skill-creator validation, native plugin validator, portable schemas/links/manifests, generated-bundle drift and whitespace checks passed.
  • Copied setup-bundle tests cover preview/cancellation without writes, save-off, synthetic shadow activation, revocation and project opt-out without a source tree or sibling quest skill.
  • Isolated native Codex/Claude installations and skills installers 1.5.25/1.7.0 passed for both skills, including explicit bundle and repository discovery, exact bytes/modes and installed tools after source removal.
  • No paid Jev probes, live qualification or personal settings changes. Conversation instructions were reviewed; a live end-to-end host question-UI session was not run.

See the global routing report, wizard report, and setup skill. Most added setup-bundle content is generated from existing canonical routing tools and schemas.

@coldtea-pr-lens

coldtea-pr-lens Bot commented Sep 25, 2026 •

Copy link
Copy Markdown

◈ PR Lens

🟢 +4 new · 🟠 ~12 changed · 🔴 -0 removed · 2 flows · 43 files · commit 198ae32


Architecture

Architecture diagram for GrillerGeek/guildhall at 198ae32

16 components touched across 4 lanes.

Play the interactive walkthrough


Inside the changed components — 2 views

Component view — Conversational setup wizard and config persistence

The conversational setup wizard, policy resolution engine, and durable approval ledger.

Architecture view of Component view — Conversational setup wizard and config persistence in GrillerGeek/guildhall

Component view — Dual-skill bundle packaging and validation

The builder and validation jobs producing independent standalone quest and routing-setup skill bundles from canonical sources.

Architecture view of Component view — Dual-skill bundle packaging and validation in GrillerGeek/guildhall

Data flow

Data flow diagram for GrillerGeek/guildhall at 198ae32

Conversational routing setup and approval flow · Quest worker dispatch routing check

Follow each request, response and payload


The other flows — 1 sequence

Quest worker dispatch routing check

Sequence diagram of Quest worker dispatch routing check in GrillerGeek/guildhall

View

  • Architecture lens
  • Data flow lens
  • Expand every detail

Tip

Set github.draw: on-demand in .github/pr-lens.yml and PR Lens stops drawing every push. Comment @pr-lens draw on a pull request when you want that one drawn

🪧 More tips
  • Run npx skills add coldteadotai/pr-lens, then tell your coding agent: "Diagram the change you just made with PR Lens and attach it to the pull request."
  • Run npx @coldtea/pr-lens-cli analyze --base origin/main on a branch, then npx @coldtea/pr-lens-cli render .pr-lens/graph.json. Same lenses, your own model key, before the pull request exists
  • Untick Architecture lens or Data flow lens under View to hide a diagram, or tick Expand every detail to open every section. The comment redraws in a few seconds
  • Click the link under each diagram to open it on a canvas you can zoom, pan and step through
  • The diagrams are links. Click one to open it on the canvas, then press W or click play to walk through the change
  • Open a diagram on the canvas, then press W or click play to walk through the change one step at a time
  • The CLI's render reads .github/pr-lens.yml and applies your renames, exclusions and lane pins at draw time
  • Set github.comment.collapsed: true in .github/pr-lens.yml to fold the comment behind one View architecture and data flow row. Drawing still runs on every push
  • Add .github/workflows/pr-lens.yml with coldteadotai/pr-lens/packages/action@v0 and your model provider's key as its api-key to run PR Lens from your own CI. Any /chat/completions endpoint works
  • Push a commit and the comment redraws for the new head. A slow older run never overwrites a newer one
  • Switch GitHub to dark mode and the diagrams follow. The moving dots are this pull request's data in motion

Thanks for using PR Lens! It's built by Coldtea, free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

@GrillerGeek GrillerGeek changed the title feat(routing): global defaults and persistent approvals feat(routing): global defaults and guided setup wizard Sep 25, 2026
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