Skip to content

docs: list starfish-events in both languages - #91

Open
Herklos wants to merge 1 commit into
masterfrom
cursor/docs-events-catalog-cd8d
Open

docs: list starfish-events in both languages#91
Herklos wants to merge 1 commit into
masterfrom
cursor/docs-events-catalog-cd8d

Conversation

@Herklos

@Herklos Herklos commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The package table and trees omitted starfish-events, and the count sentence called it TypeScript-only. Both packages/ts/events and packages/python/events exist. webhook stays the only TypeScript-only package.
  • Python usage snippets in packages/python/events/README.md and website/docs/analytics/events.md imported starfish_server.router.sync_router and starfish_protocol.config. Those modules are not in the tree. The snippets now import CollectionConfig and SyncConfig from starfish_server, and SyncRouterOptions and create_sync_router from starfish_server.router.
  • CLAUDE.md lists both events packages in the repository tree.

Test plan

  • Confirmed create_sync_router and SyncRouterOptions are exported from starfish_server.router
  • Confirmed SyncConfig and CollectionConfig are exported from starfish_server
  • Confirmed starfish_server.router.sync_router and starfish_protocol.config are not modules
  • Docs only. No runtime change. No lockfile change.
Open in Web Open in Cursor 

@cursoragent @Herklos
docs: list starfish-events in both languages
e6d8a4d 
The package catalog called events TypeScript-only and left it out of the
package table and trees. Python usage snippets imported modules that are
not in the tree. Point those snippets at starfish_server and
starfish_server.router.

Co-authored-by: herklos.dev <herklos.dev@protonmail.com>
@Herklos Herklos self-assigned this Oct 13, 2026

This branch has not been deployed

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

Reviewers

No reviews

Assignees

@Herklos Herklos

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Herklos @cursoragent