refactor(orchestrator): resolve tq command from build profile - #316
Merged
Conversation
Remove the TQ_EXECUTABLE managed-copy contract and require the profile-specific tq or tqdev command on PATH. Inject tqdev reinterpretation guidance into agent prompts and document the updated startup contract.\n\nVerified with: go test ./...
Add a test-local tq executable to PATH so the migration preflight test reaches its intended failure after the new command availability check.\n\nVerified with: go test ./internal/cli/tq; go test ./...
7 tasks
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
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.
Resolve the Tasq CLI command from the build profile
Summary
Replace the
TQ_EXECUTABLEmanaged-copy contract with a build-profile-basedtq/tqdevcommand contract. The selected command must be available onPATH, and development prompts explicitly explain how to reinterprettasq-cliexamples.主な変更点:
tqand the development command astqdev; reject unsupported profiles.tq service startlaunches services and again when the Orchestrator starts.tqdevandtasq-clireinterpretation guidance into initial, resumed, and continuation prompts.やったこと
TQ_EXECUTABLEpropagation and managed CLI copying while preservingTQ_MANAGED_RUN.devbuild profile in the development Orchestrator binary.動作確認
go test ./...git diff --checknpm run buildincmd/web/frontendレビュー & 動作確認 チェックリスト
tq, development resolvestqdev, and unsupported profiles fail.PATH.tasq.task_work_promptisfalse.TQ_EXECUTABLEvalues are stripped andTQ_MANAGED_RUNremains available.推奨テスト計画:
go test ./...with the frontend assets generated.tq service startresolves the expected command fromPATH.tasq-cliguidance usetqdev.その他気になることや相談ごと
Existing
$TQ_HOME/system/bin/tq-managedfiles are intentionally left in place but are no longer created or referenced.