PRDCT-470: phase-1 dev-docs migration — one script run (165 pages)#1027
PRDCT-470: phase-1 dev-docs migration — one script run (165 pages)#1027Iamfle4ka wants to merge 5 commits into
Conversation
Extends the Jekyll->Astro converter into the full phase-1 migrator: ports every developers.keboola.com page in one run with zero hand edits after. Encodes all gaps found in the pilot ports: include inlining (beta-warning -> admonition), Jekyll literal-escape moustache, scalar redirect_from, relative image refs -> absolute public paths, non-image assets, link policy (help-domain strip, dev-domain internalize, skip-map to canonicals, documented source-typo fixes), cli -> cli/keboola-as-code remap + scripted deprecation banner, redirect_from injection into main canonicals (frontmatter-scoped, indent-matched), Developer Docs nav group, repo-wide inbound link flips (weave-PR files excluded), and MIGRATION-REPORT.md generation. Collision baseline = upstream/main tree, so re-runs are idempotent (verified: second run changes nothing). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Generated by scripts/migrate-devdocs.mjs (previous commit) from devdocs/main
@429efb38. Re-running the script reproduces this tree byte-for-byte.
- 165 pages ported 1:1 at preserved paths (automate, integrate incl. the jobs
Queue-API page, extend, overview/api + encryption); cli -> cli/keboola-as-code
('Keboola as Code CLI') with a scripted deprecation banner -> kbagent CLI.
- 10 pages skipped: already woven into help by open weave PRs #1019/#1022/#1023,
MCP canonical, dead chrome — see MIGRATION-REPORT.md.
- 3 redirect_from injected into main canonicals (mcp, database, orchestrator);
data-streams redirects ride with #1023.
- Temporary 'Developer Docs' nav group (phase 2 = topic re-placement per
PLACEMENT-MAP); repo-wide inbound dev links flipped to internal (57 links).
Verified: build clean (423 pages, 184 redirect pages), audit-phase2 0 new broken
links / 0 missing images (and heals 1 pre-existing break), script idempotent.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Why 165 of 175 — the 10 skipped pages, explainedThe skip list is part of the script (
Net effect after this PR + the three weave PRs merge: every one of the 175 dev URLs either serves its ported page or 301s to its canonical help home. |
The Developer Docs group was 6 flat links (no nesting -> no expanders/drill-down, inconsistent with the rest of the sidebar). The script now parses the dev repo's _data/navigation.yml and mirrors its full tree: Home dropped, /overview/ children promoted, SKIPped (weave-owned) nodes pruned with their children, cli URLs remapped and retitled 'Keboola as Code CLI'. 163 nav entries, every URL verified against the ported set; previous generated block is replaced on re-run (idempotent, verified). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Nav updated: the Developer Docs group now mirrors the dev site's own navigation tree 1:1 (163 nested entries — same expandable-group markup as every other section, so drill-down/chevrons/dot styling are consistent). Generated by the script from the dev repo's |
The fact-checker QA pass exposed two silent link bugs in the skip-map rewriting: (1) prefix matching against single-page skips swallowed their ported siblings and even image paths (/overview/api/#... and /overview/encryption-N.png collapsed to /overview/); (2) exact-skip rewrites dropped #anchors (/overview/#stacks -> /overview/). Now: exact match for single-page skips, prefix only for whole skipped subtrees (data-streams), anchors preserved when the target is the same URL. Corrects 22 ported pages. Audit stays at baseline; idempotency re-verified. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
fact-checker pass on
|
Marks every sidebar entry migrated from developers.keboola.com with an accent
dot (links via href-prefix ::before; groups via details:has() on their summary)
plus a legend at the bottom of the sidebar ('migrated from developers.keboola.com').
Theme-aware via --sl-color-accent; verified in a production build in both themes.
Scoped purely by URL prefix and clearly labeled TRANSITIONAL — the whole block is
removed in phase 2 together with the temporary Developer Docs group.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Transitional UI added (f4e5ecd): every sidebar entry that migrated from developers.keboola.com now carries an accent dot (pages + their groups, incl. the Developer Docs root) with a legend at the bottom of the sidebar — so the dev origin is visible at a glance while the seam still exists. CSS-only ( |
Devdocs migration chain review (#1027 → #1029 → #1031 → #1032 → #1033)Overall: high-quality, mechanically sound work. Script-generated (
Per-PR verdicts
Nits / follow-ups (none block the build)
Merge order (hard dependency)Bottom-up, one at a time: #1027 → #1029 → #1031 → #1032 → #1033. Each is stacked on the previous branch; GitHub auto-retargets each to Automated review pass (Claude Code), reviewed by a human maintainer — not an approval. Verified against the PR branches (frontmatter/slug/redirect/link scans); page prose was sampled, not exhaustively read. |
|
Nits 1–3 addressed in [#1036](#1036) — one commit stacked on #1033's branch, so it merges last. Counts at commit time were 19/9/24 (your 6/9/26 had drifted as branches moved); self-link traps and dead anchors got judgment rewrites, all listed in the PR body. Audit issues on the branch drop 56 → 13 with no new findings. Nit 4 (edge-level 301s for the retired dev domain) is tracked as a follow-up in PRDCT-560. |
Draft — phase 1 of the dev→help merge, fully script-driven (per Jordan: the migration must be a script, verifiable by validating the code — no LLM moving content by hand).
How to verify
Review commit 1 (
scripts/migrate-devdocs.mjs) — it is the complete spec of every transformation. Commit 2 is its output:reproduces the tree byte-for-byte (idempotent — a second run changes nothing; collision baseline is the
upstream/maintree).What it does
devdocs/main@429efb38:automate/,integrate/(incl. thejobs/Queue-API page),extend/(84 pp),overview/api/+overview/encryption/;cli/→cli/keboola-as-code/("Keboola as Code CLI") with a scripted:::caution[Deprecated]banner → kbagent CLI (Jordan's call, Jul 15).MIGRATION-REPORT.md.{% include %}inlining (beta-warning → admonition), Jekyll literal-escape moustache, scalarredirect_from, relative image refs → absolute public paths, CSV/ZIP assets, kramdown/liquid syntax, one documented source-typo link fix (brabch→branch)./integrate/mcp/→/ai/mcp-server/,/integrate/database/→/components/extractors/database/,/integrate/orchestrator/→/flows/injected into the canonical pages (frontmatter-scoped, indent-matched). Data-streams redirects ride with PRDCT-541: merge dev data-streams into /storage/data-streams/ #1023.Verification
npm run buildclean — 423 pages, 184 redirect pages.audit-phase2: 0 new broken links, 0 missing images (heals 1 pre-existing break — the importer page now exists). The 2 remaining broken links pre-date this PR onmain.dist/; deprecation banner renders on/cli/keboola-as-code/.Supersedes
#1020 / #1021 / #1024 / #1025 (closed) — their scopes re-execute here with zero hand edits. Weave PRs #1019 / #1022 / #1023 remain separate (owner-level content merges).
Gate note: the 10 open developers-docs fix-PRs (#391–#398, #402, #383) should merge and this PR re-run before final review, so their corrections ride along.
🤖 Generated with Claude Code