docs: seed the Feature Support by Chain page for the interactive matrix#1430
docs: seed the Feature Support by Chain page for the interactive matrix#1430SahilAujla wants to merge 9 commits into
Conversation
Rewrites the stub page (same slug) with the FeatureSupportMatrix widget embed and the feature-support:auto marker block that UpdateFeatureSupportFromDaikonWorkflow maintains (AX-545). Prose outside the markers stays human-owned, exactly like the CU costs page. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_0152TGB1HprVNykx38Er8dwW
🔗 Preview Mode
|
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4dfc0e0966
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 4dfc0e0966
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Codex review: the seed commit rendered <FeatureSupportMatrix /> with no data file in the tree. The embed and its intro paragraph move to the stacked data PR so this commit is safe to deploy on its own. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
@codex review |
1 similar comment
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 41175112f2
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review Generated by Claude Code |
|
Codex Review: Didn't find any major issues. Another round soon, please! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
SahilAujla
force-pushed
the
sahilaujla/ax-545-feature-support-page
branch
from
750638a to
74b6a27
Compare
|
@codex review |
|
Codex Review: Didn't find any major issues. Breezy! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Matches the data branch: the block exists for search engines, LLMs, and markdown consumers, so the seed ships it inside the same hidden div and the section never renders visibly between the two merges. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Matches the data branch, which removes the closing pointer to the dashboard Chains page. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
…age comment The docs-site widget now server-renders the matrix data with on-demand revalidation (OMGWINNING/docs-site#345); the seed page's comment adopts the same freshness text as the generated-data branch so the stacked diffs stay conflict-free. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Co-authored-by: Cursor <cursoragent@cursor.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: da083c37dc
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Per Codex review: the seed commit deploys standalone, so the nav-listed page must not render blank between this merge and the stacked widget-and-data PR. Restores a one-line dashboard Chains link that the follow-up replaces with the widget embed. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Co-authored-by: Cursor <cursoragent@cursor.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 53911bbfaa
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Per Codex review: remove-markdown leaves {/* ... */} expression comments
intact, so implementation notes and generator markers were uploaded into
public Algolia records and could surface in search snippets. The cleanup
now strips them before indexing, which also covers the pre-existing
cu:auto markers.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e981c1f3dd
ℹ️ About Codex in GitHub
Your team has set up Codex to 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 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Per Codex review: the MDX-comment stripper ran on raw content, deleting JSX comments that are visible inside rendered code blocks (e.g. the OTP input examples in the wallets MFA guides). The stripper now skips fenced code blocks so only non-rendered comments are removed from Algolia records. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Co-authored-by: Cursor <cursoragent@cursor.com>
|
@codex review |
|
Codex Review: Didn't find any major issues. More of your lovely PRs please. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
1 participant
Overview
Seed half of the AX-545 Feature Support by Chain rewrite (stacked 1/2): rewrites the stub page (same slug,
reference/feature-support-by-chain) with the product-by-chain summary section and the{/* feature-support:auto */}marker block thatUpdateFeatureSupportFromDaikonWorkflow(OMGWINNING/dashboard#8097) maintains. The summary block ships inside a hidden div (it exists for search engines, LLMs, and markdown consumers; the interactive widget is the reader-facing view), and the old dashboard Chains callout is dropped — so this seed renders no visible content. Prose outside the markers stays human-owned, exactly like the CU costs page. Also addscontent/feature-support.jsonto.prettierignore— the generator owns its one-entity-per-line format, and prettier reflowing it would make every daily bot write a diff.Per Codex review, the
<FeatureSupportMatrix />widget embed moved to the stacked follow-up PR so it lands atomically with its data — this seed commit is safe to deploy on its own.The stacked follow-up PR carries the widget embed, the first generated data, and the summary matrix, and doubles as the accuracy audit.
Companion PRs: OMGWINNING/dashboard#8095 #8096 #8097 (automation stack), OMGWINNING/docs-site#345 (widget — must merge before this page renders the embed).
LINEAR TASK: https://linear.app/alchemyapi/issue/AX-545/feature-support-by-chain-interactive-endpoint-level-support-matrix-fed