Conversation
…nical relative paths Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
keboola-pr-reviewer-bot
left a comment
There was a problem hiding this comment.
Verdict: needs_human (risk 2/5) · profile docs
Needs human: a clean internal-link migration where every in-repo target and anchor I checked resolves, but substantive rewrites exceed the auto-approve bucket.
Concerns:
src/content/docs/storage/data-streams/data-streams.md: New *.keboola.com link added (stream.keboola.com); skill §5 flags cross-domain additions for human check.src/content/docs/storage/api/importer/index.md: Client links redirected to /storage/api/ with anchors dropped; target lacks clients-specific section.src/content/docs/components/extractors/database/index.md: Substantive edits (reworded bullets, deleted SSH-tunnel sentence) exceed the narrow auto-approve bucket.
Suggested reviewers: @keboola/docs
|
Closing with the rest of the #1027 chain per the 2026-07-30 decision — its base ( Not fully obsolete, though — flagging for salvage. These are link/anchor fixes, and 24 of the 28 files it touches exist on
The remaining 4 ( Follow-up: re-cut the 24 salvageable hunks as a small link-fix PR off |
Linear: PRDCT-560
Addresses nits 1–3 from Jordan's chain review on #1027 (thread
a6426e0a). Stacked on #1033's branch (PRDCT-554-variables-api-batch) — merge last, after the rest of the stack. One commit, content-only (28 files, +82/−83).Counts below are as measured at stack top
85897b8d; they drift from the review's numbers (~6/9/26) because branches moved since.a) 19
developers.keboola.comlinks → in-repo relative paths13 mechanical flips per the migration script's REMAP tables (canonical slugs used directly, e.g.
/extend/component/running/…not/extend/docker/running/…, since redirects drop#anchors), plus judgment rewrites:components/extractors/database/index.mdtutorial/index.md/#development-projectanchor doesn't exist on the woven home/extend/component/tutorial/components/applications/index.md#technical-differencesdoesn't exist onextend/component//extend/component/#component-typesai/ai-kit/index.md/extend/storage/data-streams/data-streams.md(×4)overview/index.md:161#regions-and-endpointsdoesn't exist/overview/api/#stacks-and-endpointsAll 6 rewritten anchors verified present in built HTML.
b) 9 pre-move
/integrate/storage*body links → canonicalMechanical →
/storage/api/(+/storage/api/configurations/for the raw<a href>intransformations/index.md). Judgment: two client links drop their anchors (/storage/api/has no headings — a future/storage/api/clients/page is noted in PRDCT-560);storage/api/index.md:25would self-link, so it was de-linked instead. No/integrate/variablesbody links exist — onlyredirect_fromfrontmatter, untouched.c) 24 absolute
https://help.keboola.comlinks → relativePure domain strips,
](https://help.keboola.com/…)form only.Intentionally untouched
index.md:32,40,404.md:16,17— editorial references to the dev site itself (tracked in PRDCT-560)storage/api/import-export/index.md:36— made-up example pathhelp.keboola.comliterals (prose examples ×3, JSONdocumentationUrl×1)redirect_from:frontmatterNit 4 (edge-level 301s for retired
developers.keboola.com) is out-of-repo — tracked as a follow-up in PRDCT-560.Verification
npm run buildclean;scripts/audit-phase2.mjstotal issues 56 → 13 vs baseline on the same branch (no new findings; remaining old-docs smells = the 3 intentional dev-site references)dist/🤖 Generated with Claude Code