Move orchestrator into monorepo and reorganize SDK#232
Merged
Conversation
The `client job status` command was a BubbleTea TUI that duplicated data already available via `client health status`. Remove the command, its `/job/status` API endpoint, SDK method, tests, and docs. This also drops the bubbletea dependency. The internal `GetQueueSummary` method is preserved as it powers health metrics and ListJobs status counts. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Contributor
|
Thank you for contributing to this project! 😊🕹️ |
The TestJobStatus integration test exercised the removed `client job status` CLI command and `/job/status` endpoint. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## main #232 +/- ##
========================================
Coverage 99.93% 99.93%
========================================
Files 200 204 +4
Lines 7771 8212 +441
========================================
+ Hits 7766 8207 +441
Misses 3 3
Partials 2 2
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
Move the orchestrator package from osapi-sdk into the monorepo at pkg/sdk/orchestrator/. Rename pkg/sdk/osapi/ to pkg/sdk/client/ to match Go naming conventions. Reorganize SDK docs and examples with proper Docusaurus sidebar categories for operations and features. Add comprehensive orchestrator documentation covering all public APIs: task functions, only-if-changed, failure recovery, guards with WhenWithReason, and plan introspection (Explain/Levels/Validate). 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
retr0h
changed the title
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
client job statusBubbleTea TUI command,/job/statusAPIendpoint, SDK
QueueStats()method, and dropbubbleteadependencyosapi-sdkinto monorepo atpkg/sdk/orchestrator/pkg/sdk/osapi/topkg/sdk/client/to follow Go namingconventions, update all imports across 137 files
sdk/client/andsdk/orchestrator/withDocusaurus sidebar categories for Operations and Features
documentation
Recovery, and Introspection (Explain/Levels/Validate)
WhenWithReason(), cross-link relatedfeature pages
Test plan
go build ./...passesjust go::unit— all unit tests passjust go::vet— lint clean (0 issues)just docs::build— Docusaurus builds successfully🤖 Generated with Claude Code