Skip to content

feat: add reproducible workspace file staging - #22

Merged
gjtorikian merged 7 commits into
mainfrom
riker/7-implement-the-missing-company-agnostic-a
Oct 16, 2026
Merged

gjtorikian merged 7 commits into
mainfrom
riker/7-implement-the-missing-company-agnostic-a

Conversation

@nicknisi

Copy link
Copy Markdown
Member

Implement the missing company-agnostic, attempt-local workspace-file staging capability in Quickstudy so a consumer experiment can reproducibly supply skill/document files without replacing the normal coding-agent adapter. Integrate it into actual attempt execution, public runtime types/exports, source/manifest identity, comparison contracts, failure handling and offline tests. Deliver a small independently reviewable harness PR-ready change; no WorkOS-specific behavior in shipped harness code.

Riker's check env QUICKSTUDY_LIVE_WORKOS=0 QUICKSTUDY_LIVE_TESTS=0 QUICKSTUDY_REQUIRE_DOCKER=0 DOCKER_HOST=unix:///nonexistent/quickstudy-offline.sock sh -c 'bun run typecheck && bun run lint && bun test' passed.

Opened as a draft by Riker (job 7).

@nicknisi nicknisi changed the title Implement the missing company-agnostic, attempt-local workspace-file st… feat: add reproducible workspace file staging Sep 28, 2026
@nicknisi

nicknisi commented Oct 4, 2026

Copy link
Copy Markdown
Member Author

1 GitHub check failing (lint-pr-title). Please fix it.

@nicknisi

nicknisi commented Oct 4, 2026

Copy link
Copy Markdown
Member Author

The title fix is already complete. Latest title-lint run 36433985821 passed; the displayed failure is the older pre-fix run 36366930742.

@nicknisi
nicknisi force-pushed the riker/7-implement-the-missing-company-agnostic-a branch from 3221c08 to 1aab179 Compare October 14, 2026 17:08
@nicknisi

Copy link
Copy Markdown
Member Author

Rebased onto main, preserving #23 and workspace-file staging. Reviewed runner integration and added combined offline regression coverage. Typecheck, lint, and all 487 offline tests pass; 23 Docker-gated tests skipped.

@nicknisi
nicknisi marked this pull request as ready for review October 15, 2026 18:12
@greptile-apps

greptile-apps Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The PR appears safe to merge; no new actionable issues were found.

Summary

Adds reproducible, attempt-local workspace files without replacing the normal agent adapter.

  • Container attempts can stage files and check paths without replacing the normal agent adapter.
  • File contents and absence checks shape run identity and comparison groups.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Capture file declaration] --> B[Build and check manifest identity]
  B --> C[Prepare fresh workspace and Git baseline]
  C --> D[Stage files and check absence]
  D --> E[Write normal adapter setup files]
  E --> F[Verify files and absence again]
  F --> G[Save placement evidence]
  G --> H[Run unchanged agent adapter]
  H --> I[Export and score]
  I --> J[Clean up]
  D -->|Failure| J
  E -->|Failure| J
  F -->|Failure| J
Loading

Reviews (4) · Last reviewed commit: "Merge origin/main into riker/7-implement..." · Reviewed by Greptile

Comment thread src/runner/execute.ts Resolved
@gjtorikian gjtorikian mentioned this pull request Oct 16, 2026
@gjtorikian
gjtorikian merged commit 8dd1fed into main Oct 16, 2026
13 checks passed
@gjtorikian
gjtorikian deleted the riker/7-implement-the-missing-company-agnostic-a branch October 16, 2026 21:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants