Skip to content

feat: add support for choosing namespace in case of multiple github connections. - #196

Merged
rohan-agrawal-cs merged 1 commit into
developmentfrom
cl-4549
Jul 20, 2026
Merged

feat: add support for choosing namespace in case of multiple github connections.#196
rohan-agrawal-cs merged 1 commit into
developmentfrom
cl-4549

Conversation

@rohan-agrawal-cs

@rohan-agrawal-csrohan-agrawal-cs commented Jul 17, 2026

Copy link
Copy Markdown
Contributor
  • Added functionality to scope queries to the selected GitHub connection namespace in BaseClass methods detectFramework and selectBranch.
  • Updated GitHub adapter to handle multiple user connections, allowing users to select a namespace or use a --namespace flag.
  • Enhanced tests to cover new namespace selection logic and ensure proper handling of GitHub connections.
  • Updated GraphQL queries and types to include namespace information for user connections.

#claude_code#
🤖 Generated with Claude Code

@rohan-agrawal-cs
rohan-agrawal-cs requested review from a team as code ownersJuly 17, 2026 09:49
@github-actions

github-actionsBot commented Jul 17, 2026

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

Harshi-Shah-CS
Harshi-Shah-CS previously approved these changes Jul 17, 2026
View reviewed changes

@Harshi-Shah-CSHarshi-Shah-CS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

tejas-contentstack
tejas-contentstack reviewed Jul 17, 2026
View reviewed changes
Comment threadsrc/types/launch.ts Outdated
tejas-contentstack
tejas-contentstack reviewed Jul 17, 2026
View reviewed changes
Comment threadsrc/graphql/queries.ts Outdated
@rohan-agrawal-cs@tejas-contentstack@claude
feat: add support for choosing namespace in case of multiple github c…
66458c3
…onnections.
- Added functionality to scope queries to the selected GitHub connection namespace in BaseClass methods detectFramework and selectBranch.
- Updated GitHub adapter to handle multiple user connections, allowing users to select a namespace or use a --namespace flag.
- Enhanced tests to cover new namespace selection logic and ensure proper handling of GitHub connections.
- Updated GraphQL queries and types to include namespace information for user connections.
#claude_code#
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-authored-by: Tejas Talekar <tejas.talekar@contentstack.com>
Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions

github-actionsBot commented Jul 17, 2026

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

Harshi-Shah-CS
Harshi-Shah-CS approved these changes Jul 17, 2026
View reviewed changes

@Harshi-Shah-CSHarshi-Shah-CS left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

tejas-contentstack
tejas-contentstack reviewed Jul 17, 2026
View reviewed changes
Comment threadsrc/adapters/base-class.test.ts
@rohan-agrawal-cs
rohan-agrawal-cs merged commit 8912d7a into developmentJul 20, 2026
6 checks passed
@rohan-agrawal-cs
rohan-agrawal-cs deleted the cl-4549 branch July 20, 2026 06:21
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tejas-contentstacktejas-contentstacktejas-contentstack approved these changes
@Harshi-Shah-CSHarshi-Shah-CSHarshi-Shah-CS approved these changes
@chhavi-mandowara-cstkchhavi-mandowara-cstkchhavi-mandowara-cstk approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@rohan-agrawal-cs@tejas-contentstack@Harshi-Shah-CS@chhavi-mandowara-cstk