Skip to content

fix: update copilot endpoint to enable /o paths#10216

Merged
pavinduLakshan merged 4 commits intowso2:nextfrom
RavindiFernando:copilot-next
Apr 29, 2026
Merged

fix: update copilot endpoint to enable /o paths#10216
pavinduLakshan merged 4 commits intowso2:nextfrom
RavindiFernando:copilot-next

Conversation

@RavindiFernando
Copy link
Copy Markdown
Contributor

@RavindiFernando RavindiFernando commented Apr 29, 2026

Purpose

$subject

Need to update copilot BE service to enable sub-org paths

Related Issues

  • N/A

Related PRs

  • N/A

Checklist

  • e2e cypress tests locally verified. (for internal contributers)
  • Manual test round performed and verified.
  • UX/UI review done on the final implementation.
  • Documentation provided. (Add links if there are any)
  • Relevant backend changes deployed and verified
  • Unit tests provided. (Add links if there are any)
  • Integration tests provided. (Add links if there are any)

Security checks

Developer Checklist (Mandatory)

  • Complete the Developer Checklist in the related product-is issue to track any behavioral change or migration impact.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 29, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ pavinduLakshan
❌ RavindiFernando
You have signed the CLA already but the status is still pending? Let us recheck it.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 29, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: CHILL

Plan: Pro

Run ID: 93a6fa84-e46f-4935-9728-1a044b94698f

📥 Commits

Reviewing files that changed from the base of the PR and between 1b3a54c and c0816f9.

📒 Files selected for processing (1)
  • .changeset/purple-trees-attack.md
✅ Files skipped from review due to trivial changes (1)
  • .changeset/purple-trees-attack.md

📝 Walkthrough

Walkthrough

Switches the copilot service endpoint host resolution in app configuration to use the resolveServerHost(false, true) variant, altering which server host/path modifiers are applied to the /api/server/v1/copilot endpoint.

Changes

Cohort / File(s) Summary
Service Endpoint Configuration
features/admin.core.v1/configs/app.ts
Changed copilot endpoint construction in getServiceResourceEndpoints to call resolveServerHost(false, true) instead of resolveServerHost(), affecting host/path modifier selection for /api/server/v1/copilot.
Release Changeset
.changeset/purple-trees-attack.md
Added a changeset file to trigger a patch release for @wso2is/console and @wso2is/admin.core.v1, noting the copilot endpoint update to include skipAuthzRuntimePath.
🚥 Pre-merge checks | ✅ 5 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete and does not follow the template. The Purpose section contains only '$subject' placeholder text and lacks actual description of the change; all checklist items remain unchecked. Replace '$subject' with a detailed description of the changes. Complete the checklist items accurately based on testing and verification performed. Check mandatory items if applicable.
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and specifically describes the main change: updating the copilot endpoint to include the skipAuthzRuntimePath parameter, which aligns with the code changes in the PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Changeset Required ✅ Passed The pull request includes a valid changeset file with correct YAML frontmatter listing both required packages with patch version updates and a clear description.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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: 0/1 reviews remaining, refill in 60 minutes.

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

Comment thread .changeset/purple-trees-attack.md Outdated
@pavinduLakshan pavinduLakshan merged commit 5aeb8af into wso2:next Apr 29, 2026
1 check was pending
@RavindiFernando RavindiFernando changed the title fix: update copilot endpoint to include skipAuthzRuntimePath parameter fix: update copilot endpoint to enable /o paths Apr 29, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.05%. Comparing base (49e1a4a) to head (2098600).
⚠️ Report is 8 commits behind head on next.

Additional details and impacted files
@@           Coverage Diff           @@
##             next   #10216   +/-   ##
=======================================
  Coverage   56.05%   56.05%           
=======================================
  Files          42       42           
  Lines        1024     1024           
  Branches      246      254    +8     
=======================================
  Hits          574      574           
  Misses        416      416           
  Partials       34       34           
Flag Coverage Δ
@wso2is/core 56.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

3 participants