Skip to content

Harden AGENTS.md resolution, advisor API, and caching - #6

Merged
dashaun merged 1 commit into
mainfrom
feat/agents-md-hardening
Sep 15, 2026
Merged

dashaun merged 1 commit into
mainfrom
feat/agents-md-hardening

Conversation

@dashaun

@dashaun dashaun commented Sep 15, 2026

Copy link
Copy Markdown
Owner

Implements recommendations from the code review:

  • Builder API for AgentsMdSystemAdvisor (old constructors deprecated)
  • AgentsMdParser -> AgentsMdReader rename
  • Marker-delimited context injection (robust to other advisors)
  • Concurrency guard on shared active-path state
  • O(n) context size accounting + opt-in cache-ttl
  • JaCoCo gate now covers the discovery package
  • Property validation for fallback-location and cache-ttl
  • Docs updated to match

- Add AgentsMdSystemAdvisor builder and deprecate telescoping constructors
- Rename AgentsMdParser to AgentsMdReader
- Replace endsWith context stripping with marker-delimited injection
- Add concurrency guard to AgentsMdActivePath
- Add incremental context size accounting and opt-in TTL cache
- Extend JaCoCo gate to the discovery package
- Reject blank fallback location and negative cache TTL
@dashaun
dashaun merged commit 63ea28d into main Sep 15, 2026
1 check passed
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