Skip to content

Keep run outputs out of the artifacts directory - #1029

Open
sox8502 wants to merge 1 commit into
mainfrom
daniel/artifacts-prompt-guidance
Open

sox8502 wants to merge 1 commit into
mainfrom
daniel/artifacts-prompt-guidance

Conversation

@sox8502

@sox8502 sox8502 commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Artifacts directories have been growing into run-output dumps (one project reached 32 GB / 827k files), making the Artifacts tab unusable. Agents were told the folder holds "durable project outputs" and orx-reports even suggested a models/ subfolder, with no rule against copying run outputs or pointing run code at it.

This tightens the agent guidance only:

  • SYSTEM_PROMPT.md: artifacts are finished deliverables the user reads; never a run's output directory; run outputs (checkpoints, logs, per-sample results, sweep dumps, archives, datasets, caches, copied repos) stay with the run and are cited with run tags.
  • orx-reports: drop the models/ suggestion; add an explicit do-not-save list; summarize raw results into one small table instead of copying them.

Test plan

  • cargo test --locked playbook/skill tests (65 passed)
  • GitHub CI (fmt, clippy, build, test) passing; Greptile 5/5
  • Start a new chat, run an experiment, ask for a report; confirm only the report, figures, and a small table land in Artifacts

@sox8502

sox8502 commented Nov 14, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai review

@greptile-apps

greptile-apps Bot commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] This guidance-only PR appears safe to merge.

Summary

This PR limits Artifacts to finished deliverables and keeps raw experiment files with their runs.

  • Agents keep run outputs outside the Artifacts directory.

Reviews (1) · Last reviewed commit: "Keep run outputs out of the artifacts di..." · Reviewed by Greptile

This branch has not been deployed

No deployments
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