Skip to content

Send $/start notification to the serial client upon startup.#23

Draft
cmaglie wants to merge 1 commit intomainfrom
rpc-startup-notification
Draft

Send $/start notification to the serial client upon startup.#23
cmaglie wants to merge 1 commit intomainfrom
rpc-startup-notification

Conversation

@cmaglie
Copy link
Copy Markdown
Member

@cmaglie cmaglie commented Dec 10, 2025

Motivation

Experimental: trying to figure out the best way to signal router readiness to the sketch.

Change description

Additional Notes

Reviewer checklist

  • PR addresses a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.

@cmaglie cmaglie self-assigned this Dec 10, 2025
@cmaglie cmaglie added the enhancement New feature or request label Dec 10, 2025
@cmaglie cmaglie marked this pull request as draft December 10, 2025 14:04
Comment thread internal/msgpackrouter/router.go Outdated
@cmaglie cmaglie force-pushed the rpc-startup-notification branch from 7ce5129 to 5d051a6 Compare December 17, 2025 14:37
@cmaglie cmaglie force-pushed the rpc-startup-notification branch from 5d051a6 to db9c98c Compare December 17, 2025 15:09
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Mar 18, 2026

CLA assistant check
All committers have signed the CLA.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants