Skip to content

Fix volunteer mode: configurable Woogles URL, proto-JSON serializatio…#444

Merged
domino14 merged 1 commit intomasterfrom
volunteer-mode-fixes
Mar 2, 2026
Merged

Fix volunteer mode: configurable Woogles URL, proto-JSON serializatio…#444
domino14 merged 1 commit intomasterfrom
volunteer-mode-fixes

Conversation

@domino14
Copy link
Owner

@domino14 domino14 commented Mar 2, 2026

…n, and stop behavior

  • Add woogles-url config key (defaults to https://woogles.io) so the URL is no longer hardcoded in volunteer.go
  • Pass the full config to WooglesClient so GCG parsing uses real lexicon paths instead of an empty config
  • Switch result serialization from binary proto to proto-JSON (protojson) in both the shell volunteer loop and the standalone worker
  • Fix JSON field names in ClaimJob response struct to use snake_case to match the actual API response
  • Add volunteerBusy flag so volunteer stop halts immediately when idle instead of always waiting for the next poll tick
  • Poll immediately on volunteer loop start rather than waiting 30 s

…n, and stop behavior

- Add `woogles-url` config key (defaults to https://woogles.io) so the
  URL is no longer hardcoded in volunteer.go
- Pass the full config to WooglesClient so GCG parsing uses real lexicon
  paths instead of an empty config
- Switch result serialization from binary proto to proto-JSON (protojson)
  in both the shell volunteer loop and the standalone worker
- Fix JSON field names in ClaimJob response struct to use snake_case to
  match the actual API response
- Add `volunteerBusy` flag so `volunteer stop` halts immediately when
  idle instead of always waiting for the next poll tick
- Poll immediately on volunteer loop start rather than waiting 30 s

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@domino14 domino14 merged commit d634f2e into master Mar 2, 2026
2 checks passed
@domino14 domino14 deleted the volunteer-mode-fixes branch March 2, 2026 05:34
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