fix(authors): exclude unpublished/deleted tutorials from author pages, rail & byline

[jung-thomas]

Problem

The #1732 author feature — author detail pages (/authors/<login>/), the "more from author" rail on tutorial pages, and byline linking — surfaces tutorials that are unpublished (status='INACTIVE' / soft-deleted in HANA) or deleted from the source repo. Reported on live PROD (e.g. an author page listing tutorials that are no longer published).

Root cause

The author-index builder groups the raw GitHub discovery set and never intersects it with the active/published catalog:

• scripts/fetch-tutorials.ts pushes an authorRows entry for every parsed tutorial (~:1050-1060), with no publication/deletion filter.
• scripts/parsers/author-index.ts buildAuthorIndex only groups by login + dedupes — no status predicate.
• The published/active signal lives in HANA (Tutorials.status), surfaced by the CAP catalog as tutorialMetas (srv/lib/build-catalog.js:24-26 → status = 'ACTIVE' or status is null). The navigator/browse already filter on it; the author builder never did.
• "Deleted" manifests two ways, both fixed by the same intersection: (a) stale .tutorial-cache/*.md for repo-deleted slugs (still processed in --regenerate), and (b) status='INACTIVE' soft-deletes.
• The layouts (authors/single.html, more-from-author.html) render author_index.json verbatim — so the fix belongs at the builder (one predicate fixes page + rail + byline consistently).

Fix

• buildAuthorIndex(rows, advocates, activeSlugs?) — skip rows whose slug.toLowerCase() ∉ activeSlugs. Fail-open when the set is empty/undefined (degraded / ALLOW_EMPTY_CAP builds → no filtering, matching the existing browse-skip guard). Lowercase compare (row slugs = mixed-case source dirs; catalog slugs = lowercase-canonical).
• writeAuthorPages threads activeSlugs through; an author whose tutorials are all inactive drops out of the index and their stale .md page is pruned.
• fetch-tutorials.ts passes new Set(tutorialMetas.map(m => m.slug.toLowerCase())) (already in scope at the call site).

Verification

• Unit tests added + green (test/unit/author-index/build-author-index.test.ts): exclusion, case-insensitive match, all-inactive author dropped, fail-open (undefined/empty). npx vitest run --project unit test/unit/author-index/ → 15 passed.
• Takes effect on the next content rebuild / fetch-tutorials run (regenerates author_index.json). No deploy performed.