Skip to content

feat: Add Contentstack Authentication toggle to project creation. - #195

Merged
chhavi-mandowara-cstk merged 2 commits into
developmentfrom
cl-4504
Jul 13, 2026
Merged

feat: Add Contentstack Authentication toggle to project creation.#195
chhavi-mandowara-cstk merged 2 commits into
developmentfrom
cl-4504

Conversation

@chhavi-mandowara-cstk

Copy link
Copy Markdown
Contributor

No description provided.

CopilotAI review requested due to automatic review settings July 10, 2026 10:56
@github-actions

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 Severity11525✅ Passed
🟡 Medium Severity10500✅ 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

ℹ️ Vulnerabilities Without Available Fixes (Informational Only)

The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:

  • Critical without fixes: 0
  • High without fixes: 5
  • Medium without fixes: 0
  • Low without fixes: 0

✅ BUILD PASSED - All security checks passed

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a new Contentstack Authentication toggle to the project-creation flow so environments can be created either restricted to the org (default) or publicly accessible (when explicitly disabled).

Changes:

  • Introduces a new --disable-cs-auth CLI flag and wires it through both GitHub and File Upload project creation flows.
  • Adds isContentstackAuthenticationEnabled to the config type and sends it as part of environment settings during project creation.
  • Adds unit tests covering prompt behavior and flag-driven disabling in both adapters.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 4 comments.

Show a summary per file
FileDescription
src/types/launch.tsExtends Launch config typing with isContentstackAuthenticationEnabled.
src/commands/launch/index.tsAdds the --disable-cs-auth flag and a usage example (also touches --response-mode help text).
src/adapters/github.tsPrompts for CS auth enablement (or disables via flag) and forwards the setting in project creation mutation input.
src/adapters/github.test.tsAdds tests for the new prompt/flag behavior in GitHub adapter flow.
src/adapters/file-upload.tsMirrors the CS auth prompt/flag behavior and forwards the setting in project creation mutation input.
src/adapters/file-upload.test.tsAdds tests for the new prompt/flag behavior in File Upload adapter flow.
.github/workflows/issues-jira.ymlRemoves the “Create Jira Ticket for GitHub Issue” workflow (appears unrelated to the PR’s stated feature).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment threadsrc/adapters/github.ts Outdated
Comment threadsrc/adapters/file-upload.ts Outdated
Comment threadsrc/commands/launch/index.ts
Comment threadsrc/commands/launch/index.ts
Harshi-Shah-CS
Harshi-Shah-CS previously approved these changes Jul 10, 2026

@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.

👍

@github-actions

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 Severity11525✅ Passed
🟡 Medium Severity10500✅ 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

ℹ️ Vulnerabilities Without Available Fixes (Informational Only)

The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:

  • Critical without fixes: 0
  • High without fixes: 5
  • Medium without fixes: 0
  • Low without fixes: 0

✅ BUILD PASSED - All security checks passed

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.

Comment threadsrc/adapters/file-upload.ts Outdated
Comment threadsrc/commands/launch/index.ts
Comment threadsrc/commands/launch/index.ts
@github-actions

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 Severity0525✅ 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

ℹ️ Vulnerabilities Without Available Fixes (Informational Only)

The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:

  • Critical without fixes: 0
  • High without fixes: 5
  • Medium without fixes: 0
  • Low without fixes: 0

✅ BUILD PASSED - All security checks passed

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 3 comments.

Comment threadsrc/adapters/github.test.ts
Comment threadsrc/adapters/github.test.ts
Comment threadsrc/adapters/file-upload.test.ts
Set the isContentstackAuthenticationEnabled environment field when
creating a new Launch project, for both the GitHub and FileUpload
flows. Contentstack Authentication is enabled by default; users can
opt out with the new --disable-cs-auth flag or by answering "no" at
the interactive prompt (which defaults to enabled).
#claude_code#
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions

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 Severity0525✅ 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

ℹ️ Vulnerabilities Without Available Fixes (Informational Only)

The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:

  • Critical without fixes: 0
  • High without fixes: 5
  • Medium without fixes: 0
  • Low without fixes: 0

✅ BUILD PASSED - All security checks passed

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 9 out of 10 changed files in this pull request and generated 1 comment.

Comment threadpackage.json

@dhruv-parekh-csdhruv-parekh-cs left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

@chhavi-mandowara-cstk
chhavi-mandowara-cstk merged commit e2ab44b into developmentJul 13, 2026
7 checks passed
@chhavi-mandowara-cstk
chhavi-mandowara-cstk deleted the cl-4504 branch July 13, 2026 18:19
Sign up for freeto 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.

6 participants

@chhavi-mandowara-cstk@Harshi-Shah-CS@rohan-agrawal-cs@dhruv-parekh-cs@Aravind-Kumar-cstk