fix: align resolver routing fixtures#619
Conversation
|
AStack/OpenClaw readiness follow-up evidence for this PR:
|
c4fde03 to
2a52cf4
Compare
|
Rebased onto upstream master Targeted verification after rebase:
Note: the temporary HOME was required because the default local agent environment has |
2a52cf4 to
7026ea2
Compare
|
Rebased onto upstream master |
|
Live downstream evidence from AStack/OpenClaw dogfood cut on 2026-05-05:
This PR remains one of the upstream-clean blockers for AStack FULL PASS; current dogfood status is intentionally custom/non-upstream-clean until merged and consumed. |
|
2026-05-06 maintainer merge packet / downstream readiness refresh:
Why this PR matters: it removes the resolver fixture custom-cut dependency and makes resolver doctor health upstream-owned instead of AStack-owned. |
|
Thanks for this contribution — and apologies for the slow triage. We did a full pass over the entire PR backlog. gbrain has moved fast, and the maintainer's larger "cathedral" rewrites have superseded a big share of community PRs: the AI gateway + recipes + user_provided_models system replaced almost all individual provider PRs; #1805 fixed the whole Postgres module-singleton class; #1542 unified the type taxonomy; #1657 the retrieval path; #1802 the doctor; and so on. We're closing this one in that cleanup — either the fix already landed on master, it duplicates another PR or merged change, or it's outside the current merge bar. Where a closed PR carried a genuinely valuable idea, we've recorded it in docs/designs/COMMUNITY_IDEAS.md so nothing good is lost (a few may graduate into TODOs). Please don't read the close as a judgment of the work — thank you for contributing. If you believe the underlying issue is still live on the latest master, reopen with a quick note and we'll take another look. 🙏 |
Summary
skills/RESOLVER.mdrows to include the shipped frontmatter triggers for routing-eval-backed skillssummarize this booktomedia-ingestfrontmatter so the existing book adversarial fixture has a structural routedata-researchVerification
bun run src/cli.ts doctor --fast --json->resolver_healthstatusokbun run src/cli.ts routing-eval --skills-dir skills --json->ok: true, 58/58 passedHOME=$(mktemp -d /tmp/gbrain-test-home.XXXXXX) env -u OPENCLAW_WORKSPACE -u OPENCLAW_HOME -u GBRAIN_SKILLS_DIR bun test test/resolver.test.ts test/routing-eval.test.ts test/check-resolvable.test.ts test/check-resolvable-cli.test.ts-> 161 pass, 0 failCloses #617.
Need help on this PR? Tag
@codesmithwith what you need.