Skip to content

Fix materialize dry-run filesystem writes - #70

Merged
RichardHightower merged 1 commit into
mainfrom
fix/issue-69-dry-run
Aug 31, 2026
Merged

Fix materialize dry-run filesystem writes#70
RichardHightower merged 1 commit into
mainfrom
fix/issue-69-dry-run

Conversation

@RichardHightower

Copy link
Copy Markdown
Contributor

Summary

  • make pkc_materialize --dry-run compute create/update decisions without writing concepts or directories
  • suppress write events, catalog refreshes, and log.md changes during previews
  • distinguish planned output with would_create / would_update, planned: true, and an explicit text-mode dry-run banner
  • add a byte-for-byte target-tree regression covering concepts, indexes, log.md, and write-events/
  • capture and complete the required repository worklog plan

Fixes #69

Verification

  • python3 tests/test_pkc.py — 98 tests passed
  • python3 scripts/pkc_validate.py --bundle sample-knowledge — 0 errors (8 existing missing-optional-catalog warnings)
  • git diff --check

@RichardHightower
RichardHightower merged commit 25fd07a into main Aug 31, 2026
4 checks passed
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.

pkc_materialize --dry-run writes concepts and write events

1 participant