claude-code - 💡(How to fix) Fix phipe: run-dir clean restructure — receipts, pkl path, remove run.jsonl [1 participants]

TL;DR

Remove vestigial run-dir artifacts and align the directory structure with the PhiProvenance-as-source-of-truth design to ensure consistency and cleanliness.

Guidance

• Remove run.jsonl writing and reading to avoid duplication with LogTable.
• Relocate provenance pkl outputs to {run_dir}/{run_id}/outputs/serialized/ to maintain a consistent directory structure.
• Delete dead code, including graph.json check, log_sink_uri field, and duplicate receipt free functions to reduce technical debt.
• Clean the GCS bucket by deleting unnecessary files and directories, such as phi_arr_bench_, phipe-test-, and phipe-bench-*, to maintain a organized storage system.

Example

No explicit code snippet is provided, but the guidance points should help in implementing the necessary changes.

Notes

The provided guidance is based on the information given in the issue and may not cover all possible scenarios or edge cases. It is essential to thoroughly test the changes to ensure they do not introduce new issues.

Recommendation

Apply the workaround by removing vestigial run-dir artifacts and aligning the directory structure with the PhiProvenance-as-source-of-truth design, as this will help to ensure consistency and cleanliness in the codebase.