Marcos Paulo
aae4d42229
Adds the agent-facing workspace and a 20-task plan for migrating the site to Astro. Nothing here implements the refactor; these are briefs, rules and templates that the task agents read. - .agents/ holds context, rules, checklists, skills, specialist agents, component/page/config templates and gate scripts. It is vendor-neutral so MiniMax, Gemini and Codex can all read it; CLAUDE.md just points at AGENTS.md. - .husky/ plus .lintstagedrc.json wire the three gate tiers. gate.sh locks on the shared git-common-dir so parallel worktrees serialise, and guards the assertion count in scripts/verify.mjs against a coverage drop. - plans/astro-refactor/ carries the phase graph, per-task briefs and the model-routing recommendation. These files must be tracked before fanning out: a worktree only checks out tracked files, so an untracked plan is invisible to every agent working in one. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
64 lines
2.6 KiB
Markdown
64 lines
2.6 KiB
Markdown
# Task 16 — Review desk
|
|
|
|
**Agent**: `page-migrator` · **Model**: **Codex** — highest defect risk
|
|
**Depends on**: 06, 11, 13 · **Parallel with**: 15
|
|
**Worktree**: `.agents/scripts/worktree.sh start 16 page-review-desk`
|
|
|
|
## Goal
|
|
|
|
`/skills-review/` → Astro. The most interactive page: search, filtering,
|
|
lazy file fetching, markdown rendering, a client-side diff view, six URL params,
|
|
and a live API call to `vote-service/`.
|
|
|
|
## The six query params are a public contract
|
|
|
|
`?author=`, `?skill=`, `?view=`, `?file=`, `?compare=`, `?render=` — documented
|
|
in the page footer and shared externally. All six must round-trip, and browser
|
|
back/forward must restore state (`syncUrl` / `selectFromUrl` today).
|
|
|
|
**Test every one manually.** A snapshot diff cannot catch a broken deep link.
|
|
|
|
## Islands
|
|
|
|
The catalog + detail pane is genuinely interactive: `client:load` is justified
|
|
here. The vote widget is `client:visible`. Everything else server-renders.
|
|
|
|
## vote-service integration
|
|
|
|
`window.SKILLS_REVIEW_VOTE_API = 'https://ai-for-dummies-vote.marcospaulo.dev.br'`
|
|
is set inline in `index.html` today. Either keep the global or move it to
|
|
`PUBLIC_VOTE_API` — **if you move it, update `vote-service/README.md` in the
|
|
same change**, since it documents the coupling.
|
|
|
|
`ALLOWED_ORIGIN` on the deployed service is
|
|
`https://netcracker.pages.marcospaulo.dev.br`. If the site's origin changes, the
|
|
vote API breaks with a CORS error. It does not change in this plan — but verify
|
|
after cutover.
|
|
|
|
## Asserted by verify.mjs
|
|
|
|
Interaction tokens: `from './catalog.js'`, `from './files.js'`, `renderList`,
|
|
`renderDetail`, `selectSkill`, `packageSummary`, `markdownHeadings`,
|
|
`markdownToc`, `loadSelectedFile`, `schedulePackageSearch`, `fetchSource`,
|
|
`packageSearchText`, `diffMarkup`, `diffRows`, `searchParams.set('compare')`,
|
|
`markdownMarkup`, `syncUrl`, `selectFromUrl`, `URLSearchParams`,
|
|
`navigator.clipboard`, `document.execCommand`.
|
|
|
|
Plus the catalog count: `id:'` occurrences across both catalogs **must equal 24**.
|
|
|
|
Same rule as task 15 — re-point with task 19, never delete.
|
|
|
|
## Watch for
|
|
|
|
The markdown renderer is hand-rolled (`markdownMarkup`, `markdownHeadings`,
|
|
`markdownToc`). Task 06 moves rendering to build time — but the **diff view
|
|
needs raw source text**, not rendered HTML. Keep both available.
|
|
|
|
## Done when
|
|
|
|
- [ ] All six query params round-trip; back/forward restores state
|
|
- [ ] Search, filter, file tabs, preview, change lens, download, copy all work
|
|
- [ ] Vote widget reaches the live API; CORS preflight succeeds
|
|
- [ ] Snapshot diff empty; screenshots match; checklist complete
|
|
- [ ] `npm run gate` green
|