Skip to content

Improve Copilot code reviews with repository-specific guidance - #1315

Merged
ebma merged 1 commit into
stagingfrom
codex/improve-copilot-code-reviews
Aug 5, 2026
Merged

Improve Copilot code reviews with repository-specific guidance#1315
ebma merged 1 commit into
stagingfrom
codex/improve-copilot-code-reviews

Conversation

@ebma

@ebma ebma commented Aug 5, 2026

Copy link
Copy Markdown
Member

Summary

  • add a concise repository-wide checklist that prioritizes concrete correctness, security, contract, and regression findings
  • add scoped API/payment-engine guidance for validation, concurrency, monetary precision, idempotency, corridor coverage, integrations, and migrations
  • add scoped React and Solidity guidance for async state races, amount handling, state ownership, signature/replay safety, token accounting, and deployment invariants

Why

Recent Copilot reviews repeatedly found valuable issues in these categories, sometimes only on a later pass. GitHub documents that reviews are non-deterministic and context-limited, so the instructions focus each review on Vortex-specific invariants instead of asking generically for an exhaustive review.

Validation

  • bunx prettier --check .github/copilot-instructions.md .github/instructions/api.instructions.md .github/instructions/react.instructions.md .github/instructions/solidity.instructions.md
  • parsed every path-specific frontmatter block as YAML and verified applyTo is present
  • git diff --cached --check

@netlify

netlify Bot commented Aug 5, 2026

Copy link
Copy Markdown

Deploy Preview for vortex-sandbox ready!

Name Link
🔨 Latest commit e1d8b0b
🔍 Latest deploy log https://app.netlify.com/projects/vortex-sandbox/deploys/6a7334c98dcc6500090ae01c
😎 Deploy Preview https://deploy-preview-1315--vortex-sandbox.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Aug 5, 2026

Copy link
Copy Markdown

Deploy Preview for vortexfi ready!

Name Link
🔨 Latest commit e1d8b0b
🔍 Latest deploy log https://app.netlify.com/projects/vortexfi/deploys/6a7338f2dbda7041e065eef1
😎 Deploy Preview https://deploy-preview-1315--vortexfi.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@netlify

netlify Bot commented Aug 5, 2026

Copy link
Copy Markdown

Deploy Preview for vrtx-dashboard ready!

Name Link
🔨 Latest commit e1d8b0b
🔍 Latest deploy log https://app.netlify.com/projects/vrtx-dashboard/deploys/6a7334c92a74d300085ce857
😎 Deploy Preview https://deploy-preview-1315--vrtx-dashboard.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@ebma
ebma merged commit b470bb7 into staging Aug 5, 2026
6 checks passed
@ebma
ebma deleted the codex/improve-copilot-code-reviews branch August 5, 2026 13:32
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.

1 participant