Repository navigation
ci(codex): update search index and block tags - #14
Conversation
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThis CI-generated pull request refreshes the Codex index artifacts from commit 54c66d9, changing indexed content and report metadata while removing one search entry; review should focus on whether the generated outputs match the source revision and workflow expectations. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Warning Review limit reached
This review includes 3 billable files and costs up to $0.75.
Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing. Or wait 47 minutes for your next included review. View limit details
Comment |
PR Summary by QodoRefresh Codex search artifacts and block-tagging metadata
AI Description
Diagram
High-Level Assessment
Files changed (3)
|
|
Guards passed on 4901545:
Labelled |
Code Review by Qodo
1.
|
The postings came from the index's content field, a document's first 5,000 characters, so a term past that point never found its document, the cut kept a fragment of the word it fell in as a term, and an edit near a guide's top pushed terms out of search: the index of #14 no longer finds docs/DEVELOPMENT.md by architecture or sqlite3, nor docs/RECATALOG_GUIDE.md by generated or reviewing. buildPostings reads each listed document under the root as scripts/auto-index.js parses it, the body after the frontmatter for Markdown and the whole file for YAML. A document it cannot read there, or whose path leads outside the root, keeps the content field's postings, with a warning naming it. The embeddings read the same text as before.
3167841 to
4901545
Compare
The search index files built from 8041bea by run https://github.com/framerslab/codex/actions/runs/39584174987:
codex-index.json,codex-search.json,codex-report.jsonandcodex-blocks.json.master is protected, so the index reaches it through this pull request. The Build and Publish Codex Index workflow opens and updates it with
FRAMERS_BUMP_TOKEN, so its Test Suite and rename-guard run;landing-follows-the-bump-checks.ymllabels itlandingonce both pass on its head, and it lands after its review comments are read, answered and resolved. A later run replaces this branch with its own build, and closes the pull request when the index no longer differs from master's.Summary by Sourcery
CI: