Skip to content

fix(prompt): retry a failed registry prompt fetch on the next run - #709

Closed
LinuxDevil wants to merge 4 commits into
mainfrom
obs/o10-prompt-fail-closed
Closed

LinuxDevil wants to merge 4 commits into
mainfrom
obs/o10-prompt-fail-closed

Conversation

@LinuxDevil

Copy link
Copy Markdown
Owner

Traces sent to a local observe daemon had empty inputs and outputs because content capture is opt-in, so Agent Forge couldn't show why a run failed. Agents built with createAgent now capture message content by default when LOUSHO_OBSERVE points at a loopback collector (127.0.0.1, localhost, ::1). captureContent: false turns it off; non-loopback collectors are unchanged. Docs state plainly that redactContent does not scrub message content.

Ticket: O6 (Forge Observe vs LangSmith evaluation follow-up)

Changes

  • test(observe): make the loopback redaction test falsifiable; clarify redactContent docs
  • feat(observe): capture message content by default on a local collector

Verification: focused vitest, tsc --noEmit, eslint on changed files (details in the review package); full suite and fallow run in CI.

🤖 Generated with Claude Code

LinuxDevil and others added 4 commits October 24, 2026 17:13
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…redactContent docs

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…as no fallback

createAgent({ prompt: 'name@latest' }) with no instructions and an unreachable
daemon (or 404, or LOUSHO_OBSERVE unset) ran with no system prompt while the log
claimed the embedded instructions were used. It now rejects the first run with
SDKError LOUSHO_PROMPT_UNRESOLVED; with instructions set, behaviour and the
warning text are unchanged.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A failed resolution was cached for the agent's lifetime, so with fail-closed a
daemon briefly down at build made every later run reject. A failure is now
dropped so the next run retries; successes stay cached. Adds retry, env-ref
and specific-message tests, splits agentInstructions to keep complexity down,
and notes the retry in docs/observability.md.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@LinuxDevil

Copy link
Copy Markdown
Owner Author

Opened from the wrong branch by mistake (orchestration race); O6 is reopened from obs/o6-capture-loopback and O10 will get its own PR after review.

@LinuxDevil LinuxDevil closed this Oct 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant