Expose bench-available deployed definitions through one target-discovery API (CL-7351) - #551
Draft
TheGreatAxios wants to merge 7 commits into
Draft
Expose bench-available deployed definitions through one target-discovery API (CL-7351)#551TheGreatAxios wants to merge 7 commits into
TheGreatAxios wants to merge 7 commits into
Conversation
TheGreatAxios
force-pushed
the
cl-7351-target-discovery
branch
from
September 2, 2026 06:08
3b5b2f5 to
12d3a9b
Compare
TheGreatAxios
changed the base branch from
cl-7349-contracts
to
cl-7350-routine-target
September 2, 2026 06:08
TheGreatAxios
force-pushed
the
cl-7351-target-discovery
branch
from
September 2, 2026 07:44
12d3a9b to
5fabdf8
Compare
TheGreatAxios
force-pushed
the
cl-7351-target-discovery
branch
from
September 2, 2026 07:58
5fabdf8 to
2f6f0c6
Compare
Hub launcher and scheduler resolve the routine's asset to its newest approved definition at fire time and fail closed via reportError; routines-tools imports wire shapes from @corbits/routines/client; web, chat, hub-client, evals, and e2e callers rename the field.
…efinitions (CL-7351) Adds GET /api/tenants/:tenantId/workflows/targets, the single list every routine-authoring surface reads. listLaunchableDefinitions selects the newest authored, deployed, frozen workflow_definition row per asset in the tenant; listRoutineTargets authorizes each row for the acting principal with @intx/authz before anything is counted or returned, applies the catalog product filter, classifies a single-step conversational fold as an agent, and pages by an opaque cursor over (name, definitionAssetId). The web app's client-side purpose-definitions filtering over the raw definitions listing is replaced by listRoutineTargets / listAllRoutineTargets from @corbits/routines/client.
TheGreatAxios
force-pushed
the
cl-7351-target-discovery
branch
from
September 2, 2026 08:30
2f6f0c6 to
588273f
Compare
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.
Linear: https://linear.app/abklabs/issue/CL-7350
Builds to docs/workflow-model.md (CL-7349): a routine stores its target workflow asset id and resolves the newest deployed, frozen definition at create and (to be wired) at launch.
Status — incomplete, opened early on a budget cut
Done (packages/routines only): schema + migration 0006 (backfill, delete unresolvable, drop definition_id), src/target.ts resolver + typed rejections, routes/workflow-routine-routes/client/store/drafts cutover to definitionAssetId, unit + DB-gated tests. Package typechecks and its 291 tests pass (DB suites skipped locally — Docker/Postgres unavailable).
NOT done (repo-wide typecheck currently fails; pushed with --no-verify):
Resolves CL-7351