Skip to content

fix(go-core): compile runners suite helpers - #215

Merged
rowan-stein merged 3 commits into
mainfrom
noa/fix-go-core-runners-llm-helpers
Jul 17, 2026
Merged

fix(go-core): compile runners suite helpers#215
rowan-stein merged 3 commits into
mainfrom
noa/fix-go-core-runners-llm-helpers

Conversation

@casey-brooks

Copy link
Copy Markdown
Contributor

Summary

  • Include llm_gateway_helpers_test.go in the svc_runners go-core E2E build tags.
  • Fixes runners PR fix(chat-app): remove cluster admin membership #70 E2E compile failure where main_test.go references shared LLM gateway helper functions while running with svc_runners only.

Unblocks agynio/runners#70 and agynio/architecture#161.

Validation

  • PATH=/root/.nix-profile/bin:/root/go/bin:$PATH buf generate in suites/go-core - passed
  • CGO_ENABLED=0 PATH=/root/.nix-profile/bin:$PATH go test -tags 'e2e svc_runners' ./tests/... in suites/go-core - compile passed; runtime failed dialing runners:50051 because no local runners service is running

@noa-lucent noa-lucent left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Reviewed the small E2E fix. The runners tag now includes the shared LLM gateway helper file required by main_test.go, and the reusable run-tests action can check out an explicit e2e ref so downstream PRs can exercise suite changes before merge.

Approving from code review.

@rowan-stein
rowan-stein merged commit 4fe22dd into main Jul 17, 2026
1 check passed
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