Skip to content

fix(config): add empty URL and circular group validation#13

Merged
lostmygithubaccount merged 1 commit intomainfrom
cody/validate-empty-urls-and-circular-groups
Apr 6, 2026
Merged

fix(config): add empty URL and circular group validation#13
lostmygithubaccount merged 1 commit intomainfrom
cody/validate-empty-urls-and-circular-groups

Conversation

@lostmygithubaccount
Copy link
Copy Markdown
Member

Summary

  • Add validation that URL entries cannot have empty URL strings (both simple and full forms)
  • Add detection of self-referencing groups (e.g., dev = ["dev"])
  • Add detection of indirect circular group references (e.g., a = ["b"], b = ["a"])
  • 7 new tests covering all new validation paths

🤖 Generated with Claude Code

Close two validation gaps: reject URL entries with empty strings and
detect self-referencing or mutually-referencing groups in validate().

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@lostmygithubaccount lostmygithubaccount enabled auto-merge (squash) April 6, 2026 00:23
@lostmygithubaccount lostmygithubaccount merged commit 642cc16 into main Apr 6, 2026
1 check passed
@lostmygithubaccount lostmygithubaccount deleted the cody/validate-empty-urls-and-circular-groups branch April 6, 2026 00:28
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