Skip to content

Normalize run config payload for determinism#41

Merged
WhatsYourWhy merged 2 commits into
mainfrom
codex/locate-and-fix-important-codebase-bug
Jan 28, 2026
Merged

Normalize run config payload for determinism#41
WhatsYourWhy merged 2 commits into
mainfrom
codex/locate-and-fix-important-codebase-bug

Conversation

@WhatsYourWhy

Copy link
Copy Markdown
Owner

Motivation

  • Ensure run metadata and manifest/ledger hashes are deterministic across different output directories by removing non-semantic output_dir from the serialized run config payload.

Description

  • Drop output_dir from the run config payload in run() by adding config_payload.pop("output_dir", None) to determinant/run.py so manifests do not incorporate environment-specific paths.

Testing

  • Ran pytest -q, which failed during collection with ModuleNotFoundError: No module named 'determinant' (tests were not able to execute to completion).

Codex Task

@WhatsYourWhy WhatsYourWhy merged commit 1d85861 into main Jan 28, 2026
2 checks passed
@WhatsYourWhy WhatsYourWhy deleted the codex/locate-and-fix-important-codebase-bug branch January 28, 2026 08:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant