Skip to content

fix(schemas): allow spaces in Azure DevOps project and repo names#1170

Merged
brendan-kellam merged 3 commits intomainfrom
claude/issue-1166-20260504-1755
May 4, 2026
Merged

fix(schemas): allow spaces in Azure DevOps project and repo names#1170
brendan-kellam merged 3 commits intomainfrom
claude/issue-1166-20260504-1755

Conversation

@brendan-kellam
Copy link
Copy Markdown
Contributor

@brendan-kellam brendan-kellam commented May 4, 2026

Update the regex patterns for projects and repos in the Azure DevOps connection schema to accept names containing spaces. Azure DevOps natively supports spaces in project and repository names, but the previous pattern [\w.-]+ excluded them.

Fixes #1166

Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Azure DevOps project and repository names with spaces are now accepted in configurations.
    • Added three audit events for organization membership lifecycle: member added, member removed, member left.
    • Per-user JWT session versioning: removed users’ JWT cookies, personal API keys, and OAuth tokens are invalidated on next request.

Update the regex patterns for `projects` and `repos` in the Azure DevOps
connection schema to accept names containing spaces. Azure DevOps natively
supports spaces in project and repository names, but the previous pattern
`[\w.-]+` excluded them. The org segment keeps its original restriction
since Azure DevOps organization names are URL-based and don't support spaces.

Fixes #1166

Co-authored-by: Brendan Kellam <brendan-kellam@users.noreply.github.com>
@github-actions

This comment has been minimized.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 4, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e49284c7-e2bc-40f8-be87-9a28ef4adc0f

📥 Commits

Reviewing files that changed from the base of the PR and between 4f484c6 and 7094932.

📒 Files selected for processing (1)
  • CHANGELOG.md

Walkthrough

The PR relaxes Azure DevOps schema regexes to allow spaces in project and repository name segments by changing [\w.-]+ to [\w. -]+ in the TypeScript schema source, generated JSON schema, and docs snippet; the CHANGELOG was updated accordingly.

Changes

Azure DevOps Schema Validation Pattern Update

Layer / File(s) Summary
Schema Pattern Source
packages/schemas/src/v3/azuredevops.schema.ts
Regexes for properties.projects.items and properties.repos.items changed to permit spaces in project/repo name segments ([\w. -]+ replacing [\w.-]+).
Generated Schema
schemas/v3/azuredevops.json
Generated JSON schema patterns updated to match the loosened regexes allowing spaces in project and repo segments.
Documentation Snippet
docs/snippets/schemas/v3/azuredevops.schema.mdx
Docs snippet adjusted so projects and repos item pattern values reflect the new regexes permitting spaces.
Changelog
CHANGELOG.md
Unreleased “Fixed” entry added noting the Azure DevOps connection schema now permits spaces in project and repository names.

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Out of Scope Changes check ⚠️ Warning The CHANGELOG.md includes unrelated entries for three new EE audit actions (org.member_added, org.member_removed, org.member_left) that are not connected to the Azure DevOps schema fix or issue #1166. Remove the EE audit action entries from CHANGELOG.md as they are outside the scope of the Azure DevOps spaces fix, or address them in a separate PR.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: updating the Azure DevOps schema to allow spaces in project and repo names, which matches the code changes across all three schema files.
Linked Issues check ✅ Passed The PR successfully addresses issue #1166 by updating the Azure DevOps schema patterns to accept spaces in project and repository names, enabling the native integration instead of requiring workarounds.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/issue-1166-20260504-1755

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.

Comment @coderabbitai help to get the list of available commands and usage tips.

@mintlify
Copy link
Copy Markdown

mintlify Bot commented May 4, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
sourcebot 🟢 Ready View Preview May 4, 2026, 6:11 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@brendan-kellam
Copy link
Copy Markdown
Contributor Author

@claude add a changelog

@claude
Copy link
Copy Markdown

claude Bot commented May 4, 2026

Claude finished @brendan-kellam's task in 41s —— View job


Changelog Added

  • Read CHANGELOG.md to understand format
  • Add entry under [Unreleased]
  • Commit and push

Added a ### Fixed entry under [Unreleased] in CHANGELOG.md:

### Fixed
- Fixed Azure DevOps connection schema to allow spaces in project and repository names. [#1170](https://github.com/sourcebot-dev/sourcebot/pull/1170)

Co-authored-by: Brendan Kellam <brendan-kellam@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@schemas/v3/azuredevops.json`:
- Line 56: The current pattern string "^[\\w.-]+\\/[\\w. -]+$" allows leading or
trailing spaces in the project/repo segments; replace that pattern with a
stricter regex that requires each segment to start and end with a non-space
character (e.g., begin with a word/digit/dot/hyphen and disallow
leading/trailing spaces) so names like " My Project" or "My Project " are
rejected, and apply the same replacement to the other identical pattern
occurrence in the file.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 154889cd-bb02-41e0-a989-0121a1e785ec

📥 Commits

Reviewing files that changed from the base of the PR and between 1bac95b and 0d67d56.

📒 Files selected for processing (3)
  • docs/snippets/schemas/v3/azuredevops.schema.mdx
  • packages/schemas/src/v3/azuredevops.schema.ts
  • schemas/v3/azuredevops.json

Comment thread schemas/v3/azuredevops.json
@brendan-kellam brendan-kellam merged commit 2719903 into main May 4, 2026
6 of 7 checks passed
@brendan-kellam brendan-kellam deleted the claude/issue-1166-20260504-1755 branch May 4, 2026 18:18
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.

[bug] Azure DevOps connection schema rejects project and repo names containing spaces

1 participant