Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 9, 2026

Implementation Plan

  • Analyze existing test files and identify violations of AGENTS.md guidelines
  • Fix sol-anchor/idl_test.go:
    • Replace assert.Nil(t, err) with require.NoError(t, err) (10 instances)
    • Add t.Helper() to readFromFile helper function
    • Replace manual error checking in readFromFile with require.NoError(t, err)
  • Fix helper functions missing t.Helper():
    • Add t.Helper() to loadProjectFromState in evm-events-calls/generate_test.go
    • Add t.Helper() to launchLocalServer in tests/integration_test.go
    • Add t.Helper() to runTestsInDocker in tests/integration_test.go
    • Add t.Helper() to runTestLocally in tests/integration_test.go
    • Add t.Helper() to validateBinary in tests/integration_test.go
    • Add t.Helper() to runCommand in tests/integration_test.go
  • Run tests to verify changes don't break anything
  • Request code review
Original prompt

Implement this and create PR

The user has attached the following file paths as relevant context:

  • AGENTS.md

Created from VS Code.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@maoueh
Copy link
Contributor

maoueh commented Jan 9, 2026

@copilot The task is to implement the plan attached which is plans/plan-fix-list-choices-ordering-comprehensive.md

@maoueh maoueh closed this Jan 9, 2026
@maoueh maoueh deleted the copilot/update-agents-documentation branch January 9, 2026 15:10
Copilot AI requested a review from maoueh January 9, 2026 15:10
Copilot stopped work on behalf of maoueh due to an error January 9, 2026 15:10
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