Skip to content

fix(core): suppress noisy warning for inline llm-grader prompts#903

Merged
christso merged 1 commit intomainfrom
fix/901-inline-prompt-warning
Apr 1, 2026
Merged

fix(core): suppress noisy warning for inline llm-grader prompts#903
christso merged 1 commit intomainfrom
fix/901-inline-prompt-warning

Conversation

@christso
Copy link
Copy Markdown
Collaborator

@christso christso commented Apr 1, 2026

Summary

  • Added looksLikeFilePath() heuristic to detect inline prompts vs file paths
  • Skips file resolution entirely for strings that are clearly inline prompts (multi-line, >200 chars, 3+ words)
  • Only warns when string looks like a file path but wasn't found

Test plan

  • 8 unit tests for looksLikeFilePath() covering file extensions, relative/absolute paths, multi-line, long strings, natural language, empty strings
  • All 1731 tests pass
  • Build, typecheck, lint clean

Closes #901

🤖 Generated with Claude Code

@christso christso force-pushed the fix/901-inline-prompt-warning branch from e50ce04 to 3e65ab1 Compare April 1, 2026 21:13
@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Apr 1, 2026

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0a9ab26
Status: ✅  Deploy successful!
Preview URL: https://0f70ed63.agentv.pages.dev
Branch Preview URL: https://fix-901-inline-prompt-warnin.agentv.pages.dev

View logs

@christso christso force-pushed the fix/901-inline-prompt-warning branch 2 times, most recently from 92ca542 to 0d2d961 Compare April 1, 2026 21:42
Add explicit file:// prefix for prompt file references.
Bare strings are always treated as inline text — no file resolution attempted.

Closes #901

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@christso christso force-pushed the fix/901-inline-prompt-warning branch from 0d2d961 to 0a9ab26 Compare April 1, 2026 21:52
@christso christso merged commit 5d0040b into main Apr 1, 2026
4 checks passed
@christso christso deleted the fix/901-inline-prompt-warning branch April 1, 2026 21:54
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.

fix: suppress noisy 'file not found' warning for inline llm-grader prompts

1 participant