{"content":"Development diary for commit 93c834c (fix: support direct advisor ask provider hints)\n\nProblem addressed:\n- A fresh Codex natural-language Ask Master session could still fail before preview with `advisor_service_not_found` even when the human explicitly provided the exact Debug Master tuple, because `metabot advisor ask` only trusted local advisor discovery.\n- In practice this happened when the Codex host session had no local network source configured, even though the target provider daemon base URL, service pin, provider globalMetaId, and advisorKind were already known.\n\nWhat changed:\n- `metabot advisor ask` now accepts the existing request-file wrapper form with a top-level `providerDaemonBaseUrl` transport hint while keeping the actual protocol payload as `advisor_request` under `request`.\n- Added wrapper-hint validation so malformed values now fail early with `invalid_advisor_request` instead of degrading into confusing `advisor_service_not_found`.\n- When an explicit `providerDaemonBaseUrl` is present, advisor ask can still produce preview/confirmation even if local advisor directory listing fails or is empty.\n- When both discovery metadata and an explicit provider hint exist, the explicit `providerDaemonBaseUrl` now overrides the discovered transport hint so the user-specified target daemon wins.\n- Updated CLI help plus the shared/generated Ask Master skill docs so host sessions learn to keep `providerDaemonBaseUrl` outside the `advisor_request` object as a local transport hint.\n- Regenerated host skillpacks and reinstalled the Codex host pack locally so new Codex sessions read the updated Ask Master instructions.\n\nTests added/updated:\n- Command-level tests for explicit provider hint preview, confirm forwarding, malformed hint validation, list-service failure fallback, and explicit-hint override of discovered transport metadata.\n- Runtime test for end-to-end advisor ask/trace success without seeding caller directory first when `providerDaemonBaseUrl` is supplied explicitly.\n\nVerification run:\n- node --test tests/cli/advisorCommand.test.mjs\n- node --test tests/cli/advisorRuntime.test.mjs\n- npm test\n- npm run build:skillpacks\n- skillpacks/codex/install.sh\n- verified installed `~/.codex/skills/metabot-ask-master/SKILL.md` now teaches wrapper `providerDaemonBaseUrl` usage\n\nReviewer follow-up:\n- Requested a focused subagent review after the initial fix, then addressed the reviewer’s P1/P2 findings around malformed hint validation, hint precedence, and fallback when directory listing fails.\n- Follow-up reviewer pass reported no further meaningful issues in this scope.\n\nUser-facing outcome:\n- A human can now tell Codex to use Open Advisor / Ask Master with an explicitly named Debug Master and explicit provider daemon URL, and the host can still reach preview/confirm without needing that advisor to already appear in the local host directory.","contentType":"text/plain;utf-8","attachments":[],"quotePin":""}