Repository navigation
Add private DGX task activity and execution archives (JVNAUTOSCI-2751) - #646
Merged
Merged
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
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.
Merge decision: ready for code publication — the existing worker launch path exposes private task-linked activity before exit and a verified downloadable archive after exit in a bounded canonical-service fixture. Live activation and the two selected historical backfills remain an operator handoff; this PR does not establish that recording is already active on the DGX.
User outcome
JVNAUTOSCI-2751: delegated coding tasks gain a private run history and readable/downloadable execution record instead of only host paths and a final prose result.
Material changes
The new read boundary prevents a concrete disclosure: normal attachment upload grants organisation visibility, while coding records may contain private conversation context. Run reads/downloads instead retain the original authorised audience and recheck source access.
Evidence
launch()fixture subprocess exposes records through the Flask task route before exit, then the requester downloads the reconciled ZIP. Coverage includes large outputs, partial lines, finalisation retry, source corruption, redaction, source-access revocation, reassignment, legacy-header denial and real Flask-session/trusted-controller context resolution.Ship boundary
The bounded worker-path fixture and targeted scope/recovery checks support this code merge. Silent loss/corruption, audience expansion, duplicate/reassigned attachment or false archive completion would block it.
Live controller/backend/UI activation, shared blob-store read-back and the requested selected backfills remain outstanding. The retained DGX browser bridge points at another checkout and requires operator retargeting for authenticated candidate acceptance. Jira connector access failed with reauthentication required, so delivery status must be reconciled by the coordinator.
No hidden reasoning is extracted or decrypted. Summaries are preserved only when emitted in the exec stream. No supported bounded rollout export is configured; raw session rollouts are not collected. Oversized capture remains explicitly pending with local originals retained. Broad historical import, a new polling schedule and unrelated deployment are outside this change.