Skip to content

docs: update AGENTS.md with script testing and env var guidelines#2462

Merged
maxandersen merged 1 commit into
mainfrom
docs/update-agents-md
May 14, 2026
Merged

docs: update AGENTS.md with script testing and env var guidelines#2462
maxandersen merged 1 commit into
mainfrom
docs/update-agents-md

Conversation

@maxandersen
Copy link
Copy Markdown
Collaborator

Updates AGENTS.md with lessons learned from #2459:

  • Script consistency: note that behavior must be consistent across curl/wget/PowerShell, with a concrete example of tool-specific quirks (curl --retry-delay 0 vs wget --waitretry=0)
  • Env var checklist: new env vars need defaults in all three scripts + documented in the reference table in installation.adoc
  • Script testing: run real scripts from src/main/scripts/, not synthetic copies or extracted functions — use env var overrides to point at WireMock
  • Conditional tests: concrete assumeTrue examples for shell availability

@maxandersen maxandersen merged commit 2d4f1f5 into main May 14, 2026
23 of 28 checks passed
@maxandersen maxandersen deleted the docs/update-agents-md branch May 14, 2026 09:11
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