Repository navigation
AIGTWY-4878 [1/3] Extract existing native transcript helpers - #968
Merged
Merged
Conversation
tt-le
added this pull request to stack #969
October 2, 2026 19:37
tt-le
changed the title
tt-le
marked this pull request as ready for review
October 2, 2026 19:58
tt-le
force-pushed
the
tt-le/aigtwy-4878-agent-test-helpers
branch
3 times, most recently
from
October 6, 2026 13:44
67ce51e to
d2edc55
Compare
tt-le
changed the title
david-siqi-liu
approved these changes
Oct 6, 2026
tt-le
force-pushed
the
tt-le/aigtwy-4878-agent-test-helpers
branch
from
October 7, 2026 02:57
d2edc55 to
8a42c7e
Compare
xsh310
pushed a commit
to xsh310/ucode
that referenced
this pull request
Oct 7, 2026
…abricks#965) ## Summary Catalog discovery consumer, stacked on the behavior-preserving helper extraction databricks#968. Uses the preconfigured CUJ3 workspace; provisioning databricks#964 remains separate. - Require Claude's Sonnet/Haiku/Kimi set and Codex's GPT Luna/Kimi set under `ug_e2e.models`; exclude Gemini and `ug_e2e.other_models` decoys from scoped discovery and native pickers. - Inspect gateway catalogs and native pickers, then complete separate default and explicit-model tasks through real TUI and headless agent commands. - Use the shared dedicated-workspace `cuj` fixture, authenticated SDK client, and isolated local sessions. Catalog reads use SDK GET calls; workspace configuration remains read-only. - Run through the existing `dedicated-cuj` CI job. Remove the custom catalog lane and bespoke integration-runner mode; shared workflow and conftest match current main. ## Stack Rebased onto current main through databricks#968. MCP registration databricks#990 is the child consumer stack; its changes are not included here. ## Validation - 133 focused runner/evidence/model-discovery/provider-catalog tests passed. - All 10 catalog cases collect; consumer Ruff lint/format and `git diff --check` pass. - No new live inference pass. Fresh SP credentials and a clean disposable runner are still required. - Previously observed Claude default-banner/picker failures remain unresolved; migration does not weaken those assertions. - Claude model evidence is response-reported; Codex evidence joins its client-selected model to the completed turn. Neither proves the gateway's backing destination. Skills discovery remains deferred.
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
2 participants
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
Behavior-preserving extraction only: move existing Claude/Codex answer parsing, session paths, and child-session detection into per-agent modules. Shared evidence callers and existing journeys retain their behavior and names.
Removed
tests/test_integration_agent_helpers.py: the existing public evidence tests already cover these paths. No new model attribution, catalog requests, HTTP helpers, picker behavior, managed-config readers, or CUJ CI changes belong to this PR.Stack
Targets main; rebased onto current main. Catalog discovery #965 targets this branch, and MCP registration #990 targets #965. Provisioning #964 remains parked separately.
Validation
git diff --checkpassed; commit/push secret scans passed.