molecule-ai/molecule-core
36
0
Fork 2
Code Issues 17 Pull Requests 21 Actions 1.2k Packages Projects Releases Wiki Activity

fix(delegation_test): remove stray '}' breaking go vet (closes #257) #260

Closed
claude-ceo-assistant wants to merge 1 commits from fix/core-257-delegation-test-stray-brace into revert/core-123-plugin-drift-detector
pull from: fix/core-257-delegation-test-stray-brace
merge into: molecule-ai:revert/core-123-plugin-drift-detector
Conversation 2 Commits 1 Files Changed 1 -1

1 Commits

Author SHA1 Message Date
dev-lead
b25debb0a6 fix(delegation_test): remove stray '}' breaking go vet (closes #257)
Some checks failed
Secret scan / Scan diff for credential-shaped strings (pull_request) Successful in 11s
Details
sop-tier-check / tier-check (pull_request) Failing after 10s
Details
audit-force-merge / audit (pull_request) Has been skipped
Details 
Commit 97768272 ("test(delegation): add isDeliveryConfirmedSuccess helper")
left a stray closing brace on line 1265, immediately after the proper
function-closing brace on 1264. Result:

    delegation_test.go:1265:1: expected declaration, found '}'

`go vet ./internal/handlers/...` halts at this syntax error before any
type-checking begins, masking other latent issues in the package.

This is the THIRD silent main-breaker found in this debugging session,
all from missing build/vet gates on PRs (see internal#219 — CI hardening
RFC).
 2026-05-10 01:16:36 -07:00