Skip to content

Persist agreed PR Decisions across independent runs - #41

Merged
dkropachev merged 1 commit into
mainfrom
fix/34-persistent-decisions
Oct 27, 2026
Merged

dkropachev merged 1 commit into
mainfrom
fix/34-persistent-decisions

Conversation

@dkropachev

Copy link
Copy Markdown
Contributor

Issue

Agreed PR review choices disappear when independent runs use disposable data directories. Later reviews can repeat settled questions, and concurrent refreshes can overwrite locally recorded choices (#34).

Closes #34.

Fix

Add DK_PR_DECISIONS_HOME as the exclusive absolute root of the complete Decisions cache, scoped by GitHub host, base repository and PR. Keep the current location when unset and reject invalid or repository-local stores without fallback. Code Factory supplies persistent storage and access separately.

Hold a per-PR process lock across refresh, analysis, modification and publication bookkeeping. Synchronize private atomic replacements and preserve uncertain-publication receipts through interruption. Keep offline list free of writes and local persistence separate from GitHub publication.

Retain original agreement evidence and replacement history across restarts and source edits or deletion. Record agreed PR-level choices immediately through add or replace in review and iteration workflows, then refresh live context and engineering preparation. Temporary context imports and unselected suggestions remain temporary; recovery grants no preparation readiness, review coverage, publication authority or iteration budget.

Document storage seeding and migration, preserve existing cache formats and API v2 records, and synchronize plugin versions at 1.3.0.

Allow hosts to share the complete Decisions cache through an exclusive
DK_PR_DECISIONS_HOME root while preserving the existing default location.
Serialize each modifying PR operation and synchronize atomic replacements.
Retain exact agreement evidence, replacement chains and uncertain
publication receipts across disposable-run cleanup and source changes.

Record agreed choices immediately in review and iteration procedures.
Document recovery and migration, preserve temporary imports, and cover
independent processes, concurrent changes, interrupted writes and
preparation invalidation. Synchronize plugin versions at 1.3.0.

Closes #34.
@dkropachev
dkropachev merged commit da7d512 into main Oct 27, 2026
1 check passed
@dkropachev
dkropachev deleted the fix/34-persistent-decisions branch October 27, 2026 04:04
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.

Persist agreed PR review decisions across independent runs

1 participant