Skip to content

fix(copilot): restore v1 tool manifest compatibility - #178

Merged
TradingGoose-Dev merged 1 commit into
mainfrom
staging
Jul 28, 2026
Merged

fix(copilot): restore v1 tool manifest compatibility#178
TradingGoose-Dev merged 1 commit into
mainfrom
staging

Conversation

@BruzWJ

@BruzWJ BruzWJ commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Restore the Copilot runtime tool manifest version to v1, matching the service’s strict contract.
  • Update the manifest test to assert v1.
  • Remove the project-specific header from the AGPL license text.

Why

The Copilot service strictly validates the manifest version and rejects chat requests when Studio advertises a mismatched version. This restores compatibility.

Affected Areas

  • apps/tradinggoose
  • apps/docs
  • packages/*
  • Workflows / execution
  • Realtime / sockets
  • Market data / charting
  • Dev tooling / CI / infra
  • Documentation only
  • Other: Repository licensing text

Validation

bun run test -- lib/copilot/runtime-tool-manifest.test.ts
git diff --check origin/main...staging

@vercel

vercel Bot commented Jul 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
studio (staging) Ready Ready Preview, Comment Jul 28, 2026 4:30pm

Request Review

@greptile-apps

greptile-apps Bot commented Jul 28, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Restores Copilot tool-manifest compatibility with the service’s strict version contract.

  • Changes the runtime tool-manifest version from v2 to v1.
  • Updates the corresponding manifest test assertion.

Confidence Score: 5/5

The PR appears safe to merge, with the implementation and test consistently restoring the documented Copilot manifest version contract.

The changed runtime constant is propagated directly into Copilot chat requests, the test now verifies the same value, and no repository consumer requires or branches on the previous v2 value.

Important Files Changed

Filename Overview
apps/tradinggoose/lib/copilot/runtime-tool-manifest.ts Restores the manifest version literal to v1 without changing the tool payload.
apps/tradinggoose/lib/copilot/runtime-tool-manifest.test.ts Updates the contract test to assert the restored v1 manifest version.

Reviews (1): Last reviewed commit: "fix(copilot): revert tool manifest versi..." | Re-trigger Greptile

@TradingGoose-Dev
TradingGoose-Dev merged commit dfbe3d9 into main Jul 28, 2026
7 checks passed
@TradingGoose-Dev
TradingGoose-Dev deleted the staging branch July 28, 2026 16:42
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.

2 participants