Skip to content

fix: reject blank agent selectors - #583

Open
intellectronica wants to merge 1 commit into
mainfrom
review/blank-agent-selectors-581
Open

intellectronica wants to merge 1 commit into
mainfrom
review/blank-agent-selectors-581

Conversation

@intellectronica

Copy link
Copy Markdown
Owner

Fixes #581

Summary

  • Rejects empty and whitespace-only default_agents entries during TOML config loading.
  • Rejects empty and whitespace-only [agents.*] table keys before agent config mapping.
  • Adds regression tests for both selector paths.

Verification

  • npx jest tests/unit/core/ConfigLoader.test.ts --runInBand --coverage=false
  • npm run check:package-lock
  • npm run format:check
  • npm run lint
  • npm test
  • npm run build
  • npm audit --omit=dev --audit-level=moderate
  • npm pack --dry-run

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.

Reject blank TOML agent selectors

1 participant