docs: explain portable UI routing and project team adoption - #4
Conversation
Assisted-by: Codex:GPT-6 [OpenSpec, Git]
Assisted-by: Codex:GPT-6 [Git, OpenSpec, Node.js, Playwright]
Assisted-by: Codex:GPT-6 [Git, GitHub CLI]
![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: f5a7c2eb4f
ℹ️ 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".
|
|
||
| function categoryFor(name) { | ||
| return CATEGORY_OF.get(name) || 'Other'; | ||
| return uiEntries.has(name) ? 'UI & UX' : CATEGORY_OF.get(name) || 'Other'; |
There was a problem hiding this comment.
On Windows, node:path.relative() returns values such as prometheus-ui-ux\SKILL.md, while the caller derives dirName with rel.split('/'). Consequently, this new lookup never receives a catalog ID, so all 40 UI skills fall into Other and the advertised UI & UX catalog page is not generated on a supported native-Windows host. Normalize separators or derive the directory with path.dirname/path.sep before this lookup.
AGENTS.md reference: AGENTS.md:L34-L36
Useful? React with 👍 / 👎.
| - The Boss: `pnpm docs:index`, `pnpm docs:check-structure`, `pnpm docs:check-frontmatter` and `pnpm docs:check-index` passed. The stale release-guide source reference was corrected to IntegrationPage.tsx after the frontmatter gate identified it. | ||
| - The Boss README rendered through the installed Markdown renderer. Its tracked build/logo.png loaded at natural width 1024; header, product links and upstream attribution were inspected. | ||
| - `openspec validate ui-ux-documentation-refresh --strict` passed. | ||
| - Fresh-context independent source review returned no findings. The REST gateway could not choose a distinct configured reviewer, so the harness-native fallback was used; no cross-model independence is claimed. |
There was a problem hiding this comment.
This Codex-authored commit adds substantial generated documentation and explicitly records an agent review, but its raw commit message contains no Assisted-by: AGENT:MODEL [tools] trailer. Add the required trailer so the generated content remains attributable under the repository's certification policy.
AGENTS.md reference: AGENTS.md:L28-L28
Useful? React with 👍 / 👎.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
1 participant
Documents the merged UI/UX routing and project-team adoption for mini: README, installation/team guides, platform limits, and the Docusaurus UI catalog/navigation. Inventory now distinguishes 97 mini skills, 40 portable UI entries, and the 41-entry shared catalog. Commands distinguish proposal-only export, project installation, and unexecuted phase-boundary evidence requirements.
Local validation:
npm run build:deployinsite/passed; the new guide rendered from the production build; OpenSpec strict validation passed. Independent fresh-context source review found no introduced defects. Windows, every-harness execution, and installed Electron acceptance remain explicitly unverified.Follow-up documentation to #3. Companion documentation PRs: Full pack, mini, The Boss. No runtime, dependency, or submodule changes; no deployment.