Skip to content

feat(pxi): add the approval-gated navigation.goTo operation - #15411

Draft
cephalization wants to merge 1 commit into
pxi-tool-subsumptionfrom
pxi-navigation-operation
Draft

feat(pxi): add the approval-gated navigation.goTo operation#15411
cephalization wants to merge 1 commit into
pxi-tool-subsumptionfrom
pxi-navigation-operation

Conversation

@cephalization

Copy link
Copy Markdown
Contributor

resolves #

Implement the RFC from #15339 on top of the subsumption machinery. The
handler registers at the app root and is available from every page —
its job is to be reachable when the operation a script wants is not.

- path validates against the registered route catalog with matchPath,
  so parameterized routes work and the model cannot navigate to a guess
- reason is a required schema field, rendered in the approval card
  ("PXI wants to take you to Playground — '…'")
- never auto-accepts: bypass edit mode still shows the card
- accept resolves only after the route change commits; a navigation the
  router refuses (blocker/unsaved work) resolves { ok: false }
- rejection resolves { ok: false } with a do-not-retry, offer-a-link
  instruction, so the script continues gracefully
- dispatch's not-mounted error and the execute_ui instructions now
  point at ui.navigation.goTo, closing the self-healing loop
- unit tests cover validation, parameterized matching, already-there,
  accept/blocked/declined, and the no-bypass rule

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@cephalization
cephalization force-pushed the pxi-navigation-operation branch from 52a747f to b7b785a Compare August 12, 2026 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 📘 Todo

Development

Successfully merging this pull request may close these issues.

1 participant