Skip to content

fix(results): ignore unrelated files during remote sync#1347

Open
christso wants to merge 2 commits into
mainfrom
session/age-12
Open

fix(results): ignore unrelated files during remote sync#1347
christso wants to merge 2 commits into
mainfrom
session/age-12

Conversation

@christso

Copy link
Copy Markdown
Collaborator

Summary

Remote results sync now stays scoped to AgentV result artifacts. If the results repo has unrelated local files such as package.json or package-lock.json, the dashboard sync path no longer treats them as metadata errors; it pulls remote updates when possible, stages only .agentv/results/**, and pushes only commits whose changed paths are still result artifacts.

This preserves the safety guard against pushing non-result files while allowing normal run/result sync to continue after accidental local junk appears in the results repo.

Verification

  • Red-path evidence from dogfood: POST /api/projects/financial-research-agent/remote/sync returned block_reason: "Results repo has non-results working tree changes" with dirty_paths: ["package-lock.json", "package.json"]; dashboard screenshot saved at /tmp/agentv-sync-error.png.
  • bunx biome check packages/core/src/evaluation/results-repo.ts packages/core/test/evaluation/results-repo.test.ts
  • bun test packages/core/test/evaluation/results-repo.test.ts — 16 pass
  • bun --filter @agentv/core typecheck
  • Pre-push hook also ran workspace typecheck and biome check . successfully.

Post-Deploy Monitoring & Validation

  • Watch dashboard/API logs for /api/projects/:projectId/remote/sync responses containing block_reason, last_error, or sync_status: "dirty".
  • Healthy signal: projects with unrelated untracked files in the results repo still return clean/synced status after safe result artifacts are pulled/pushed.
  • Failure signal: sync is blocked solely by non-result untracked files, or remote repos receive commits containing paths outside .agentv/results/**.
  • Validation window: first dogfood sync after deploy; owner: AgentV maintainer running the remote-sync dogfood bead.

Compound Engineering
Codex

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 10, 2026

Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9f4b228
Status: ✅  Deploy successful!
Preview URL: https://a31e372e.agentv.pages.dev
Branch Preview URL: https://session-age-12.agentv.pages.dev

View logs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant