fix(lychee): commit:HEAD → TBD on star-count entry (closes #338) - #340
Merged
Conversation
The star-count fork-changes entry (id: cli-* star-count, 2026-05-28) still carried `commit: HEAD`, which renders [`HEAD`](.../commit/HEAD) into README + FORK_CHANGELOG + llms-full — a broken link lychee doesn't exclude (only /commit/TBD is excluded). Flip it to the TBD placeholder, same as every other not-yet-resolved entry; maintain-fork-changes.py resolves TBD->real SHA at ship. Together with the doobidoo/mcp-memory-service exclude (e98450f) this clears the full lychee liability tracked in MemPalace#338 — no real commit/HEAD markdown links remain in any rendered doc (verified), the only residual `commit/HEAD` text is a code-span in FORK_CHANGELOG prose describing this very resolution flow, which lychee doesn't parse as a link. check-docs all 7 green; ruff check + format clean; render-docs/llms-full --check in sync. Regenerated docs committed. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #339 (the doobidoo exclude, merged as 721ea49). #339 cleared the 404 but the squash-merge landed before this companion commit, so the
commit:HEADhalf of the MemPalace#338 lychee liability is still on main.The star-count fork-changes entry (2026-05-28, "refresh upstream MemPalace star count") carried
commit: HEAD, which renders[HEAD](https://github.com/techempower-org/mempalace/commit/HEAD)into README.md + FORK_CHANGELOG.md + llms-full.txt. That URL is not a real commit and is not in lychee.toml's exclude list (only/commit/TBDis). Flipped it to theTBDplaceholder — the documented convention every other unresolved entry uses;maintain-fork-changes.pyresolvesTBD→real SHA at ship-prep.Verified locally: no real
commit/HEADmarkdown links remain in any rendered doc (the only residualcommit/HEADtext is a double-backtick code span in FORK_CHANGELOG prose describing this very resolution flow — lychee parses rendered markdown and does not extract code-span URLs). check-docs all 7 green; ruff check + format clean; render-docs / render-llms-full--checkin sync; regenerated docs committed.Closes #338 (the doobidoo exclude in #339 + this HEAD→TBD flip together clear the full lychee liability).
🤖 Generated with Claude Code