Skip to content

chore: sync FPF spec from ailev/FPF (2026-05-17)#134

Open
github-actions[bot] wants to merge 1 commit into
mainfrom
chore/sync-fpf-46d5d585
Open

chore: sync FPF spec from ailev/FPF (2026-05-17)#134
github-actions[bot] wants to merge 1 commit into
mainfrom
chore/sync-fpf-46d5d585

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Auto-detected new commit on ailev/FPF main.

CI gates this merge: lint, type, tests, hosted MCP build, Vercel preview deploy, and Playwright E2E. After a two-hour review window, the sync workflow will merge this PR automatically if every reported check/status is passing.

Generated by .github/workflows/sync-fpf.yml. Failed or still-pending checks leave this PR open for manual review.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 17, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
fpf-sh Ready Ready Preview, Comment May 18, 2026 7:18am

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 25e6b7aa64

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

"slug": "causal-use-counterfactual-support-repair",
"title": "Causal-use / counterfactual-support repair",
"staticPath": "/generated/routes/route_causal-use-counterfactual-support-repair"
"slug": "boundary-unpacking-and-claim-decomposition",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Update route references after renaming boundary route

Changing the committed route slug here means the canonical ID is now route:boundary-unpacking-and-claim-decomposition, but the rest of the repo still hard-codes route:boundary-unpacking-claim-routing (for example src/runtime/query-helpers.ts only applies the boundary claim intent scorer to the old ID, and docs/start-here.md, docs/mcp-recipes.md, and several query/MCP tests still reference the old selector). In contexts where users ask about API/contract/protocol boundary unpacking or follow the docs' route: selector, the renamed route no longer receives the intended exact/intent handling, so route retrieval and tests drift from the published snapshot.

Useful? React with 👍 / 👎.

@github-actions github-actions Bot force-pushed the chore/sync-fpf-46d5d585 branch from 25e6b7a to 762f930 Compare May 19, 2026 07:08
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 762f93018c

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

"slug": "admissible-comparison-pool-selection-selected-set-publication",
"title": "Admissible comparison / pool / selection / selected-set publication",
"staticPath": "/generated/routes/route_admissible-comparison-pool-selection-selected-set-publication"
"slug": "admissible-comparison-candidate-pool-policy-selection-and-selected-set-publication",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Update the documented comparison route selector

With this registry update, the comparison route exposed by the current published snapshot is now route:admissible-comparison-candidate-pool-policy-selection-and-selected-set-publication / route:admissible-comparison-pool-selection-or-selected-set-publication, and route:lawful-comparison-pool-selection-selected-set-publication no longer appears in published/current/fpf-index/snapshot.json. I checked the repo references and docs/start-here.md:30 still tells users to copy the old route:lawful-comparison-pool-selection-selected-set-publication selector, so following the Start Here table now sends users to a missing route instead of the comparison route.

Useful? React with 👍 / 👎.

@github-actions github-actions Bot force-pushed the chore/sync-fpf-46d5d585 branch from 762f930 to a99a320 Compare May 20, 2026 07:07
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a99a320b68

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

"slug": "causal-use-counterfactual-support-repair",
"title": "Causal-use / counterfactual-support repair",
"staticPath": "/generated/routes/route_causal-use-counterfactual-support-repair"
"slug": "boundary-unpacking-and-claim-decomposition",
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Update boundary route overrides to renamed route ID

Renaming the route slug here changes the canonical ID to route:boundary-unpacking-and-claim-decomposition, but src/runtime/answer-projector.ts still keys route-specific acceptance and next-move text on route:boundary-unpacking-claim-routing (see the switch cases around lines 186 and 204). After this sync, boundary-route answers fall through to the generic branch, so users no longer get the tailored boundary decomposition guidance that this route previously provided.

Useful? React with 👍 / 👎.

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.

0 participants