Skip to content

LINBEE-30104 | fix: one measurements call per repo - #26

Merged
nivSwisa1 merged 1 commit into
mainfrom
LINBEE-30104-one-repo-per-call
Nov 9, 2026
Merged

nivSwisa1 merged 1 commit into
mainfrom
LINBEE-30104-one-repo-per-call

Conversation

@nivSwisa1

Copy link
Copy Markdown
Contributor

A two-repo task batched both ids into one measurements call, which the scoped API token refuses with a 403; the skill now makes one call per repo and stops reading such a 403 as an expired token.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings November 9, 2026 09:09
@upwind-code-us

upwind-code-us Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

Upwind Upwind IaC Scan - ✅ Passed

0 newly introduced misconfigurations · 0 resolved · 0 total in this PR vs main

View full analysis in Upwind Console →

Scan completed in 11s

Scan history (1 scan)
Commit Scanned at New Resolved Net
e66b5e4 < 2026-11-09 09:09 UTC 0 0 0

Last scanned: e66b5e4 · 2026-11-09 09:09 UTC

@upwind-code-us

upwind-code-us Bot commented Nov 9, 2026

Copy link
Copy Markdown

Upwind Upwind Secrets Scan

0 newly introduced secrets · 0 resolved · 1 total in the files this PR changed vs main

Total breakdown: 🔶 1 PII

A secret that has been pushed remains in the repository's history even after the line is removed.

Scan completed in 15s.


Scan history (1 scan)
Commit Scanned at New Resolved Net
e66b5e4 < 2026-11-09 09:09 UTC 0 0 0

Last scanned: e66b5e4 · 2026-11-09 09:09 UTC

@upwind-code-us

upwind-code-us Bot commented Nov 9, 2026

Copy link
Copy Markdown

Upwind Upwind SAST Scan - ✅ Passed

✅ No SAST findings detected

View full analysis in Upwind Console →

Scan completed in 23s

Scan history (1 scan)
Commit Scanned at New Resolved Net
e66b5e4 < 2026-11-09 09:09 UTC 0 0 0

Last scanned: e66b5e4 · 2026-11-09 09:09 UTC

@orca-security-us orca-security-us Bot 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.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed OSS Licenses high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

Copilot AI 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.

🟢 Approved

No active review issues are reported for the per-repository measurements and scope-related 403 handling updates.

No open issues 🎉

What changed in this PR

Updates the agentic-advisor skill to avoid scope-related 403 errors when gathering LinearB health signals for tasks spanning multiple repositories.

Changes:

  • Requires one measurements request per repository.
  • Instructs the agent to retry reshaped requests in the documented format rather than treating their 403 responses as expired tokens.
  • Documents the fix and bumps the plugin version to 1.5.1.
File Description
plugins/​agentic-advisor/​skills/​agentic-advisor/​SKILL.md Clarifies per-repository requests and scope-related 403 handling.
plugins/​agentic-advisor/​CHANGELOG.md Records the multi-repository measurements fix.
plugins/​agentic-advisor/​.claude-plugin/​plugin.json Bumps the plugin version to 1.5.1.

Review effort: Balanced

@MishaKav MishaKav left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@nivSwisa1
nivSwisa1 merged commit 2f185ad into main Nov 9, 2026
15 of 16 checks passed
@nivSwisa1
nivSwisa1 deleted the LINBEE-30104-one-repo-per-call branch November 9, 2026 09:37
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