fix: regenerate source-files manifest#524
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]> Claude-Session: https://claude.ai/code/session_015TeseuGD6esLcNDWPQYcz3
|
|
Ehesp
marked this pull request as ready for review
July 8, 2026 19:57
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.
Requested by Elliot Hesp · Slack thread
Fixes #
Summary
Before — the Source Code browser on the homepage fails to load certain mdx files. For example, clicking
docs/assets.mdxreturns// Failed to load docs/assets.mdx, because the generated manifest (app/src/components/homepage/source-files.json) had gone stale: it still listed files that were since renamed/removed (e.g.docs/assets.mdx, severaldocs/assets/*.png) and was missing files that have since been added.After — the manifest is regenerated so it reflects the current repository tree. Stale entries like
docs/assets.mdxare removed and current files are present, so the Source Code browser resolves and loads them correctly.How — this change is purely the output of
bun run generate:source-files(which runsapp/scripts/generate-source-files.ts). No source logic changed; only the generated manifest was updated (148 insertions, 39 deletions).Scope
app/(hosted site, MCP, Ask AI)packages/cli/packages/mdx-bundler/docs/(product documentation)Type of change
Test plan
bun run checkpasses locallybun dev, CLI command, or other relevant command)docs/(if user-facing)Regenerated via
bun run generate:source-files; confirmeddocs/assets.mdxno longer appears in the manifest and current docs mdx entries (verified a sample) resolve to real files on disk.Notes for reviewers
The manifest goes stale whenever docs/source files are added or removed. This PR just refreshes it; consider automating regeneration in CI as a follow-up so it doesn't drift again.
Generated by Claude Code