Resolve a plan file once and give cross-site scanning one owner - #172
Merged
Conversation
…ite directory scanning. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Jz7wTcYsy68jUUjcWttd1B
…an file without parsing it. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Jz7wTcYsy68jUUjcWttd1B
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Jz7wTcYsy68jUUjcWttd1B
…ts own site scan. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Jz7wTcYsy68jUUjcWttd1B
…g in the changelog. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Jz7wTcYsy68jUUjcWttd1B
Explorbot Self-RegressionCommit
Attempt details
Session analysis — basic (native): Session AnalysisIssues creation, validation, filtering, search, and detail views produced the expected visible states. Automation was unreliable around modal and filter controls, leaving some assertions incomplete. Coverage
What works
Execution Issues
|
DenysKuchma
approved these changes
Sep 2, 2026
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.
Plan files passed to
explorbot testwere resolved and parsed twice with two different search orders (the pre-config peek scanned every registered site; the load scanned the current site's plans dir), so the peek could find one file and the run another. Now the file is resolved once and the peek's result is reused.Plan.resolveFilelocates without parsing;plans-commandandExplorBot.loadPlanseach parse once.Cross-site directory scanning ("find X under every registered site's dir") was independently implemented by PR #165 (plan loading) and PR #166 (prima's
status); both now go through single owners inglobal-config.ts(findSiteWith,listSitePlanDirs), and the hardcodedSITE_PLANS_DIRcopy in thePlandomain model is gone.Behavior change: when a plan name resolves to different files in different registered sites,
explorbot test <name>now runs the file the pre-config peek found — previously peek and run could disagree.Testing:
bun test tests/unit1227 pass / 0 fail (2 newPlan.resolveFiletests),bun test boat/prima/tests135 pass / 0 fail, biome format + lint clean.🤖 Generated with Claude Code
https://claude.ai/code/session_01Jz7wTcYsy68jUUjcWttd1B