Skip to content

fix(seo): repair main's gitleaks failure; drop docs.webhook.co from IndexNow#783

Merged
choraria merged 1 commit into
mainfrom
lane/indexnow-docs-cleanup
Jul 23, 2026
Merged

fix(seo): repair main's gitleaks failure; drop docs.webhook.co from IndexNow#783
choraria merged 1 commit into
mainfrom
lane/indexnow-docs-cleanup

Conversation

@choraria

@choraria choraria commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

🔴 First: main is red, and I caused it

main is currently failing gitleaks at 74e921ed (#782). Root cause:

In #782 I put // gitleaks:allow on the lines above the INDEXNOW_KEY constant. That does nothing — the directive must sit on the same line as the finding. #782 went green only because the .gitleaksignore fingerprint covered its branch commit, and a squash-merge mints a new commit SHA, so the suppression evaporated the moment it landed.

.gitleaksignore's own header warns that fingerprints are commit-bound and break on rebase. The squash case is the same trap, and it is now written down there.

Fix: directive moved onto the line, plus a fingerprint for the squash commit now frozen in main's history. Both are needed — the directive for any future commit touching the line, the fingerprint for the one already written.

Verified with the gitleaks CLI locally, reproducing RED on the old tree first and confirming clean after — not by watching the check flip colour. I also isolated which change does the work: with the directive present my commit's diff re-adds the key line and the scan is clean, which could not happen if the same-line form didn't work.

Second: docs.webhook.co dropped from IndexNow

The docs key file shipped in #782 as a deliberate experiment — Mintlify documents .txt as Enterprise-only, so this was unknown rather than assumed. Measured after deploy:

Host GET /<key>.txt
www.webhook.co 200, serves the key ✅
docs.webhook.co 404 Asset not found

So docs cannot host a key file and apps/docs/<key>.txt was dead weight. Removed, and docs dropped from SITEMAP_FOR_HOST and the test's KEY_FILE_DIRS with a comment recording why, so nobody repeats the experiment.

Consequence worth flagging: Bing coverage for the 150 docs URLs now depends entirely on the Bing Webmaster Tools GSC import, which needs no key file. That raises its priority from optional to the only route.

Status of the submission

www.webhook.co: 36 URLs submitted, HTTP 202 (accepted). Unaffected by this PR.

Test plan

  • Local gitleaks CLI: RED before, clean after.
  • 25 tests pass; CLI refuses docs.webhook.co cleanly and www --dry-run still resolves 36 URLs.

@choraria
choraria force-pushed the lane/indexnow-docs-cleanup branch from 488c48e to 6974653 Compare July 23, 2026 14:34
TWO fixes. The first is urgent: main is currently RED on gitleaks at
74e921e (#782), and I caused it.

1. gitleaks. In #782 I put `// gitleaks:allow` on the lines ABOVE the
   INDEXNOW_KEY constant. That does nothing -- the directive must sit on
   the SAME LINE as the finding. #782 went green only because the
   .gitleaksignore fingerprint covered its BRANCH commit, and a
   squash-merge mints a NEW commit SHA, so the suppression evaporated the
   moment it landed and main went red. The file's own header warns that
   fingerprints are commit-bound; the squash case is the same trap.

   Fixed by moving the directive onto the line, and adding a fingerprint
   for the squash commit that is now frozen in main's history. Both are
   needed: the directive for any future commit that touches the line, the
   fingerprint for the one already written.

   Verified with the gitleaks CLI locally, reproducing RED on the old
   tree first and confirming clean after -- not by watching the check
   flip colour.

2. docs.webhook.co dropped from IndexNow. The key file shipped in #782 as
   a deliberate experiment, since Mintlify documents .txt as
   Enterprise-only. Measured after deploy:
     https://www.webhook.co/<key>.txt   -> 200, serves the key
     https://docs.webhook.co/<key>.txt  -> 404 "Asset not found"
   So docs cannot host a key file and apps/docs/<key>.txt was dead
   weight. Removed, and docs dropped from SITEMAP_FOR_HOST and the test's
   KEY_FILE_DIRS with a comment recording why, so nobody repeats it. Bing
   coverage for docs comes from Bing Webmaster Tools' GSC import instead.

www is unaffected and already submitted: 36 URLs, HTTP 202.
@choraria
choraria force-pushed the lane/indexnow-docs-cleanup branch from 6974653 to 489c0b1 Compare July 23, 2026 14:48
@choraria choraria changed the title fix(seo): drop docs.webhook.co from IndexNow — Mintlify cannot host the key fix(seo): repair main's gitleaks failure; drop docs.webhook.co from IndexNow Jul 23, 2026
@mintlify

mintlify Bot commented Jul 23, 2026

Copy link
Copy Markdown

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
webhook-co 🟢 Ready View Preview Jul 23, 2026, 2:50 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@choraria
choraria merged commit 7d07827 into main Jul 23, 2026
30 checks passed
@choraria
choraria deleted the lane/indexnow-docs-cleanup branch July 23, 2026 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant