Sisyphus/Sisyphus
4
0
Fork 0
Code Issues 74 Pull Requests 11 Actions 3 Packages Projects Releases Activity

chore(client): regenerate the TypeScript client from node.proto #520

Merged
admin merged 1 commits from chore/regen-client-proto into dev 2026-10-19 17:31:13 +00:00
Conversation 1 Commits 1 Files Changed 5
+168 -26
rvnminers-A-and-N commented 2026-10-19 17:00:29 +00:00
Owner
Copy Link
Copy Source

packages/client/src/gen/sisyphus/node/v1/node_pb.ts is the desktop's TypeScript client for the node's local API. Nobody had regenerated it since #391, so it was missing everything added to node.proto since, including GetJoinedPool, WatchJoinedPool, JoinedPool and JoinedPoolState (#483, #485). This PR:

  • regenerates it with the repo's own npm --prefix apps/sisyphus run proto:generate (protoc-gen-es v2.16.0, the same as before). The diff only adds exports; nothing is renamed or removed.
  • makes make proto run that step after regenerating the Go code, so editing node.proto keeps both in step. docs/development.md, docs/desktop-backend.md and packages/client/README.md say so.

Checked: npm run typecheck is clean and npm test passes (108/108) in apps/sisyphus. actionlint is clean.

#519 adds LeavePool to node.proto. Whichever of the two merges second will regenerate this file again so it includes LeavePool.

`packages/client/src/gen/sisyphus/node/v1/node_pb.ts` is the desktop's TypeScript client for the node's local API. Nobody had regenerated it since #391, so it was missing everything added to `node.proto` since, including `GetJoinedPool`, `WatchJoinedPool`, `JoinedPool` and `JoinedPoolState` (#483, #485). This PR: - regenerates it with the repo's own `npm --prefix apps/sisyphus run proto:generate` (protoc-gen-es v2.16.0, the same as before). The diff only **adds** exports; nothing is renamed or removed. - makes `make proto` run that step after regenerating the Go code, so editing `node.proto` keeps both in step. `docs/development.md`, `docs/desktop-backend.md` and `packages/client/README.md` say so. Checked: `npm run typecheck` is clean and `npm test` passes (108/108) in `apps/sisyphus`. `actionlint` is clean. #519 adds `LeavePool` to `node.proto`. Whichever of the two merges second will regenerate this file again so it includes `LeavePool`.
Sign in to join this conversation.
Reviewers
No Reviewers
excho0
Labels
Clear labels
accessibility
Barrier affecting people with disabilities
 all-platforms
Run every platform's packaging and checks on this pull request, not only the three main ones
 area:agents
MCP server, skill, and thinking on the pool
 area:ai
Planner and model providers
 area:chain
Blockchain, SISY, verification
 area:ci
Continuous integration and build workflows
 area:client
Desktop and web clients
 area:daemon
sisyphusd coordinator, worker, scheduler
 area:p2p
libp2p hosts, relays, discovery, connections between nodes
 area:project
Project planning, roadmap, and governance
 area:protocol
Protobuf schemas and RPC contracts
 area:runtime
Workload execution and environments
 area:storage
Content-addressed storage, IPFS, Kubo, Cluster, S3, names, pinning
 bug
Something isn't working
 chore
community
Repository health, templates, Discussions, contributor experience
 dependencies
Updates to actions, Go modules or npm packages
 discussion
Migrated from GitHub Discussions
 discussion/Announcements
GitHub Discussions category: Announcements
 discussion/General
GitHub Discussions category: General
 discussion/Ideas
GitHub Discussions category: Ideas
 discussion/Q&A
GitHub Discussions category: Q&A
 discussion/Show and tell
GitHub Discussions category: Show and tell
 discussion/answered
Q&A discussion with an accepted answer
 documentation
Improvements or additions to documentation
 duplicate
This issue or pull request already exists
 enhancement
New feature or request
 good first issue
Good for newcomers
 help wanted
Extra attention is needed
 i18n
Translations and right-to-left layouts
 infra
Build, CI, releases
 invalid
This doesn't seem right
 needs:decision
Waits on a decision from both owners
 needs:hardware
Waits on a machine we do not have yet
 paired
Built together by both maintainers, with co-authored commits
 question
Further information is requested
 research
Needs investigation or a design decision
 security
Sealing, keys, access control, tokens, the security model
 status:in-review
Built and in an open pull request; closes when it merges
 test
testing
Tests, test harnesses and the rehearsal
 tracking
Roadmap and meta issues
 wontfix
This will not be worked on
type
Feature
A request, idea, or new functionality (GitHub issue type)
type
Task
A specific piece of work (GitHub issue type)
No labels area:client chore documentation infra
Milestone
No items
No Milestone First release
Projects
Clear projects
No projects
Sisyphus roadmap
Done
Assignees
Clear assignees
excho0 (Excho) rvnminers-A-and-N (Austin L.) sisyphus-bot (Sisyphus CI bot)
No Assignees rvnminers-A-and-N
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Sisyphus/Sisyphus#520