-
Couldn't load subscription status.
- Fork 4
Introduce Launchpad Controller #249
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…events Implemented the events using decl merging pattern, so that event types can be colocated with their respective packages.
these defeat the point of using neverthrow.
🦋 Changeset detectedLatest commit: 02adddf The changes in this PR will be included in the next version bump. This PR includes changesets to release 6 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
1bc45df to
3b102a5
Compare
|
This doesn't include any documentation updates because there are no end-user API changes... that said it's probably worth documenting this architecture and the design decisions somewhere in the repo that isn't just code comments... |
|
Still need to actually emit events in content/monitor 🤦 I declared the event types, but didn't actually integrate them anywhere |
|
Also worth noting: keeping the major version as 0, as the controller API could change significantly in the next 2 phases. |
Phase 1: Central Controller Architecture
Phase 1 of #215
Summary
Introduces a central controller architecture that orchestrates subsystems (content and monitor) through commands and events, providing the foundation for future persistent mode operations.
🎯 What's Changed
New Controller Package (
@bluecadet/launchpad-controller)Enhanced Subsystem Integration
@bluecadet/launchpad-utilsCLI Integration
launchpad contentandlaunchpad monitorcommands to use controller🔧 Key Features
Type-Safe Events
Duck-Typed Interfaces
Task Mode Architecture
💥 Breaking Changes
None! This is a fully backward-compatible enhancement.