Skip to content

LINBEE-30104 | fix: look the repo up by its exact URL (v1.4.5) - #21

Open
nivSwisa1 wants to merge 3 commits into
mainfrom
LINBEE-30104-exact-repo-lookup
Open

nivSwisa1 wants to merge 3 commits into
mainfrom
LINBEE-30104-exact-repo-lookup

Conversation

@nivSwisa1

@nivSwisa1 nivSwisa1 commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

The skill queries /api/v1/repositories?http_url= instead of listing every repo, so it works with a limited Agentic Advisor key. Merge before linear-b/sls-public-api (agentic key scopes).

🤖 Generated with Claude Code

✨ PR Description

Purpose: Optimize repository resolution in the agentic-advisor plugin by using exact URL lookups via the API.

Main changes:

  • Updated repository lookup to use http_url query parameter instead of local filtering
  • Improved SSH remote parsing to correctly handle ports and IPv6 hosts
  • Incremented plugin version to 1.4.5 and updated documentation and changelog

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 18, 2026 08:14

@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

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

✨ PR Review

LGTM

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how

gitstream-cm[bot]
gitstream-cm Bot previously approved these changes Oct 18, 2026

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.

🟡 Changes recommended

SSH remotes with explicit ports produce incorrect lookup URLs, losing repository-specific health signals.

1 open issue
What changed in this PR

Updates Agentic Advisor repository resolution to support limited API keys.

Changes:

  • Queries repositories by exact remote URL instead of listing all repositories.
  • Removes name-based fallback and updates lookup guidance.
  • Releases version 1.4.5.
File Description
plugins/​agentic-advisor/​skills/​agentic-advisor/​SKILL.md Adds URL-filtered lookup and updates fallback guidance.
plugins/​agentic-advisor/​CHANGELOG.md Documents the lookup change.
plugins/​agentic-advisor/​.claude-plugin/​plugin.json Bumps version to 1.4.5.

Review effort: Balanced

Comment thread plugins/agentic-advisor/skills/agentic-advisor/SKILL.md
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 18, 2026 08:22

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.

🟡 Changes recommended

SSH-port normalization corrupts bracketed IPv6 addresses, preventing repository resolution for those remotes.

1 open issue
1 resolved since last review

Review effort: Balanced

Comment thread plugins/agentic-advisor/skills/agentic-advisor/SKILL.md Outdated
@gitstream-cm
gitstream-cm Bot dismissed their stale review October 18, 2026 08:27

Review dismissed

…und 2)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 18, 2026 08:29
gitstream-cm[bot]
gitstream-cm Bot previously approved these changes Oct 18, 2026

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 unresolved findings remain, and the previously reported SSH-port and IPv6 normalization issues are addressed.

No open issues 🎉

1 resolved since last review

Review effort: Balanced

@gitstream-cm
gitstream-cm Bot dismissed their stale review October 18, 2026 08:32

Review dismissed

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants