-
Notifications
You must be signed in to change notification settings - Fork 4
Version Packages #253
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
Open
Bluecadet-Bot
wants to merge
107
commits into
develop
Choose a base branch
from
changeset-release/develop
base: develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Version Packages #253
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pending changesets means there could be unreleased API changes, so we don't want to deploy the docs until those are released.
removes some incorrect info - features that were going to be added, but don't exist yet :(
…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.
ran npm audit fix
The IPC transport is the only 'required' transport - necessary for the persistent mode to work. It implements the same interface as the future transports.
Allows for checking for running instances based on the existance of a PID file.
Updates 'content' and 'monitor' command to send commands to running controller if one is found, otherwise instantiate ephemeral controller. Updates 'start' command to create persistent controller. Updates 'stop' command to stop persistent controller (in addition to killing pm2). Adds 'status' command to query running controller.
Seems like node will block as long as there's an open server, so the IPC server takes care of this.
bug caused by using a relative path import instead of module path for logmanager
we were previously searching for all valid config files... this was unecessary, as we were just using the first from the list. Now we just return the first found.
no need for the redundant duck-typing that we had previously
test started failing after 636c91c
af4c2fb to
434dd3b
Compare
d20626e to
30bf742
Compare
transport interface/context was redundant, effectively a slightly different implementation of the existing subsystem API. Now everthing is subsystems.
it was causing weird bugs with duplicate thrown errors, etc
Also only add onTerminate listener in one spot (in CLI command), then propogate exit with subsystem disconnect() calls. fixes #247
we were calling disconnect instead of shutdown, so the apps were still left orphaned
30bf742 to
cb218de
Compare
91adbef to
6ad8d81
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to develop, this PR will be updated.
Releases
@bluecadet/[email protected]
Major Changes
#262
29c2ccbThanks @claytercek! - Refactor package exports. Removed most re-exports from the index files, and added additional package export paths. Also refactored the launchpad meta package to generate export paths that match the individual packages. This updates nearly all import paths across the entire launchpad ecosystem.#268
87d5384Thanks @claytercek! - Add log level flag to CLI to control verbosity of terminal output.Minor Changes
#249
a9a2c10Thanks @claytercek! - Introduces the new@bluecadet/launchpad-controllerpackage, which provides a centralized controller architecture for Launchpad. End user APIs remain unchanged, with the controller used internally by the CLI for command execution in "task mode".#260
747dce1Thanks @claytercek! - refactor: extract content fetch pipeline into stagesExtract the fetch pipeline from LaunchpadContent into composable stage functions (setupHooks, backup, clearOldData, fetchSources, etc.) for better testability and modularity. Simplify state management with inline phase tracking. Add comprehensive tests for fetch context and stages.
This is a breaking change as it modifies the API of LaunchpadContent by adding the loadSources() method, as well as changing the fetch() and clear() methods to accept just the source IDs instead of full source objects.
#268
fb70176Thanks @claytercek! - Move file logging config and logic to controller, and terminal logging to CLI. Refactor logging to use event bus, so logs are visible across processes.Also adds a SubsystemContext type to be shared across packages.
#260
376ee60Thanks @claytercek! - Move declaration merging to utils package instead of controller package.This improves type safety when the controller package is not a dependency, such as when using content/monitor packages in isolation.
The API stays largely the same, with some minor adjustments to import paths and type exports.
#257
e254db8Thanks @claytercek! - Adds 'start' command to CLI for starting launchpad controller in 'persistent' mode. This mode opens an IPC socket, allowing subsequent CLI commands to connect to the running controller instance. The command can be launched with the-d/--detachflag to run it in the background. Phase 2 of the multi-interface controller architecture.#260
3495b26Thanks @claytercek! - Refactor monitor and content state to use Immer. This allows us to emit patch events when state changes, which are then handled by the controller package to sync state across processes (just IPC for now).Adds a new "watch" flag to the CLI status command to allow live monitoring of the controller status.
Patch Changes
#260
634fa94Thanks @claytercek! - Move LaunchpadConfig type definition from cli package to utils package, allowing for declaration merging.#269
9f47258Thanks @claytercek! - Bump dependencies with vulnerabilities#268
6f636b6Thanks @claytercek! - Refactor subsystems to be functional instead of classes. This allows for simpler logic and easier testing. No changes to the CLI, only the JS API.Updated dependencies [
a9a2c10,747dce1,2bbb15c,82dae70,634fa94,fb70176,34bc601,9f47258,6f636b6,29c2ccb,6f636b6,376ee60,74fc748,e254db8,e254db8,3495b26]:@bluecadet/[email protected]
Major Changes
#260
747dce1Thanks @claytercek! - refactor: extract content fetch pipeline into stagesExtract the fetch pipeline from LaunchpadContent into composable stage functions (setupHooks, backup, clearOldData, fetchSources, etc.) for better testability and modularity. Simplify state management with inline phase tracking. Add comprehensive tests for fetch context and stages.
This is a breaking change as it modifies the API of LaunchpadContent by adding the loadSources() method, as well as changing the fetch() and clear() methods to accept just the source IDs instead of full source objects.
#268
fb70176Thanks @claytercek! - Move file logging config and logic to controller, and terminal logging to CLI. Refactor logging to use event bus, so logs are visible across processes.Also adds a SubsystemContext type to be shared across packages.
#268
6f636b6Thanks @claytercek! - Remove tokenization logic from content backup/download/temp paths. Also update default temp and backup paths to be in.launchpad/directory.#262
29c2ccbThanks @claytercek! - Refactor package exports. Removed most re-exports from the index files, and added additional package export paths. Also refactored the launchpad meta package to generate export paths that match the individual packages. This updates nearly all import paths across the entire launchpad ecosystem.#268
6f636b6Thanks @claytercek! - Refactor subsystems to be functional instead of classes. This allows for simpler logic and easier testing. No changes to the CLI, only the JS API.Minor Changes
#260
376ee60Thanks @claytercek! - Move declaration merging to utils package instead of controller package.This improves type safety when the controller package is not a dependency, such as when using content/monitor packages in isolation.
The API stays largely the same, with some minor adjustments to import paths and type exports.
#260
3495b26Thanks @claytercek! - Refactor monitor and content state to use Immer. This allows us to emit patch events when state changes, which are then handled by the controller package to sync state across processes (just IPC for now).Adds a new "watch" flag to the CLI status command to allow live monitoring of the controller status.
Patch Changes
#249
a9a2c10Thanks @claytercek! - Introduces the new@bluecadet/launchpad-controllerpackage, which provides a centralized controller architecture for Launchpad. End user APIs remain unchanged, with the controller used internally by the CLI for command execution in "task mode".#270
2bbb15cThanks @claytercek! - Fix orphaned promises in data store#260
634fa94Thanks @claytercek! - Move LaunchpadConfig type definition from cli package to utils package, allowing for declaration merging.#269
9f47258Thanks @claytercek! - Bump dependencies with vulnerabilities#257
e254db8Thanks @claytercek! - Clear content data store when starting new fetch. Prevents stale content when refetching with persistent controller instance.Updated dependencies [
a9a2c10,634fa94,fb70176,9f47258,29c2ccb,6f636b6,376ee60,74fc748]:@bluecadet/[email protected]
Major Changes
#249
a9a2c10Thanks @claytercek! - Introduces the new@bluecadet/launchpad-controllerpackage, which provides a centralized controller architecture for Launchpad. End user APIs remain unchanged, with the controller used internally by the CLI for command execution in "task mode".#268
fb70176Thanks @claytercek! - Move file logging config and logic to controller, and terminal logging to CLI. Refactor logging to use event bus, so logs are visible across processes.Also adds a SubsystemContext type to be shared across packages.
#262
29c2ccbThanks @claytercek! - Refactor package exports. Removed most re-exports from the index files, and added additional package export paths. Also refactored the launchpad meta package to generate export paths that match the individual packages. This updates nearly all import paths across the entire launchpad ecosystem.#268
6f636b6Thanks @claytercek! - Refactor subsystems to be functional instead of classes. This allows for simpler logic and easier testing. No changes to the CLI, only the JS API.Minor Changes
#260
376ee60Thanks @claytercek! - Move declaration merging to utils package instead of controller package.This improves type safety when the controller package is not a dependency, such as when using content/monitor packages in isolation.
The API stays largely the same, with some minor adjustments to import paths and type exports.
#270
74fc748Thanks @claytercek! - Unify subsystem and transport API.#257
e254db8Thanks @claytercek! - Adds 'start' command to CLI for starting launchpad controller in 'persistent' mode. This mode opens an IPC socket, allowing subsequent CLI commands to connect to the running controller instance. The command can be launched with the-d/--detachflag to run it in the background. Phase 2 of the multi-interface controller architecture.#260
3495b26Thanks @claytercek! - Refactor monitor and content state to use Immer. This allows us to emit patch events when state changes, which are then handled by the controller package to sync state across processes (just IPC for now).Adds a new "watch" flag to the CLI status command to allow live monitoring of the controller status.
Patch Changes
#260
634fa94Thanks @claytercek! - Move LaunchpadConfig type definition from cli package to utils package, allowing for declaration merging.#261
34bc601Thanks @claytercek! - Fix invalid named pipe on windows#269
9f47258Thanks @claytercek! - Bump dependencies with vulnerabilitiesUpdated dependencies [
a9a2c10,634fa94,fb70176,9f47258,29c2ccb,6f636b6,376ee60,74fc748]:@bluecadet/[email protected]
Major Changes
#268
fb70176Thanks @claytercek! - Move file logging config and logic to controller, and terminal logging to CLI. Refactor logging to use event bus, so logs are visible across processes.Also adds a SubsystemContext type to be shared across packages.
#262
29c2ccbThanks @claytercek! - Refactor package exports. Removed most re-exports from the index files, and added additional package export paths. Also refactored the launchpad meta package to generate export paths that match the individual packages. This updates nearly all import paths across the entire launchpad ecosystem.#268
6f636b6Thanks @claytercek! - Refactor subsystems to be functional instead of classes. This allows for simpler logic and easier testing. No changes to the CLI, only the JS API.Patch Changes
#249
a9a2c10Thanks @claytercek! - Introduces the new@bluecadet/launchpad-controllerpackage, which provides a centralized controller architecture for Launchpad. End user APIs remain unchanged, with the controller used internally by the CLI for command execution in "task mode".#269
9f47258Thanks @claytercek! - Bump dependencies with vulnerabilitiesUpdated dependencies [
a9a2c10,747dce1,2bbb15c,82dae70,634fa94,fb70176,34bc601,9f47258,6f636b6,29c2ccb,6f636b6,376ee60,74fc748,87d5384,e254db8,e254db8,3495b26]:@bluecadet/[email protected]
Major Changes
#268
fb70176Thanks @claytercek! - Move file logging config and logic to controller, and terminal logging to CLI. Refactor logging to use event bus, so logs are visible across processes.Also adds a SubsystemContext type to be shared across packages.
#262
29c2ccbThanks @claytercek! - Refactor package exports. Removed most re-exports from the index files, and added additional package export paths. Also refactored the launchpad meta package to generate export paths that match the individual packages. This updates nearly all import paths across the entire launchpad ecosystem.#268
6f636b6Thanks @claytercek! - Refactor subsystems to be functional instead of classes. This allows for simpler logic and easier testing. No changes to the CLI, only the JS API.Minor Changes
#260
376ee60Thanks @claytercek! - Move declaration merging to utils package instead of controller package.This improves type safety when the controller package is not a dependency, such as when using content/monitor packages in isolation.
The API stays largely the same, with some minor adjustments to import paths and type exports.
#260
3495b26Thanks @claytercek! - Refactor monitor and content state to use Immer. This allows us to emit patch events when state changes, which are then handled by the controller package to sync state across processes (just IPC for now).Adds a new "watch" flag to the CLI status command to allow live monitoring of the controller status.
Patch Changes
#249
a9a2c10Thanks @claytercek! - Introduces the new@bluecadet/launchpad-controllerpackage, which provides a centralized controller architecture for Launchpad. End user APIs remain unchanged, with the controller used internally by the CLI for command execution in "task mode".#270
82dae70Thanks @claytercek! - Fix monitor shutdown not working properly.#260
634fa94Thanks @claytercek! - Move LaunchpadConfig type definition from cli package to utils package, allowing for declaration merging.#269
9f47258Thanks @claytercek! - Bump dependencies with vulnerabilitiesUpdated dependencies [
a9a2c10,634fa94,fb70176,9f47258,29c2ccb,6f636b6,376ee60,74fc748]:@bluecadet/[email protected]
Major Changes
29c2ccbThanks @claytercek! - Refactor package exports. Removed most re-exports from the index files, and added additional package export paths. Also refactored the launchpad meta package to generate export paths that match the individual packages. This updates nearly all import paths across the entire launchpad ecosystem.Minor Changes
#268
fb70176Thanks @claytercek! - Move file logging config and logic to controller, and terminal logging to CLI. Refactor logging to use event bus, so logs are visible across processes.Also adds a SubsystemContext type to be shared across packages.
Patch Changes
#269
9f47258Thanks @claytercek! - Bump dependencies with vulnerabilitiesUpdated dependencies [
a9a2c10,634fa94,fb70176,9f47258,29c2ccb,6f636b6,376ee60,74fc748]:@bluecadet/[email protected]
Major Changes
#268
fb70176Thanks @claytercek! - Move file logging config and logic to controller, and terminal logging to CLI. Refactor logging to use event bus, so logs are visible across processes.Also adds a SubsystemContext type to be shared across packages.
#262
29c2ccbThanks @claytercek! - Refactor package exports. Removed most re-exports from the index files, and added additional package export paths. Also refactored the launchpad meta package to generate export paths that match the individual packages. This updates nearly all import paths across the entire launchpad ecosystem.Minor Changes
#260
376ee60Thanks @claytercek! - Move declaration merging to utils package instead of controller package.This improves type safety when the controller package is not a dependency, such as when using content/monitor packages in isolation.
The API stays largely the same, with some minor adjustments to import paths and type exports.
#270
74fc748Thanks @claytercek! - Unify subsystem and transport API.Patch Changes
#249
a9a2c10Thanks @claytercek! - Introduces the new@bluecadet/launchpad-controllerpackage, which provides a centralized controller architecture for Launchpad. End user APIs remain unchanged, with the controller used internally by the CLI for command execution in "task mode".#260
634fa94Thanks @claytercek! - Move LaunchpadConfig type definition from cli package to utils package, allowing for declaration merging.#269
9f47258Thanks @claytercek! - Bump dependencies with vulnerabilities#268
6f636b6Thanks @claytercek! - Refactor subsystems to be functional instead of classes. This allows for simpler logic and easier testing. No changes to the CLI, only the JS API.@bluecadet/[email protected]
Patch Changes
9f47258Thanks @claytercek! - Bump dependencies with vulnerabilities