Archive the last two completed OpenSpec changes - #2031
Merged
Conversation
Apply the redirect-shared-temp-to-session-scratch delta to session-cwd and tool-approval-gates. The OpenSpec CLI applied both files. Merge the surface-reminder-schedule-failures delta into netclaw-scheduling by hand. PR #2024 already corrected that requirement against the merged code, so a programmatic apply would undo those corrections. Keep the corrected text and add the scheduling-failure content: - An execution failure and a scheduling failure share one ConsecutiveFailures count. - A successful reschedule alone does not reset the count. - An unattended reschedule site emits ReminderScheduleFailed at Warning severity, and ReminderAutoDisabled at Critical severity on the threshold. - The set_reminder create path and update path return the error to the caller and emit no alert. Drop three stale delta claims. The merged code disables a reminder. It does not set a "paused" status and it does not write tasks.json. ReminderExecutionActor has no ExecutionTimeoutSeconds constant, so the one-hour absolute limit stands.
Move each change to openspec/changes/archive/2026-08-19-<name>/. The code of both changes is merged on dev. - surface-reminder-schedule-failures: PR #1886. - redirect-shared-temp-to-session-scratch: PRs #1890, #1925, #1926, #1945, and #1956. Tick task 4.3 of redirect-shared-temp-to-session-scratch. The full pr_validation suite ran green on Windows against current dev. Leave the other boxes unticked. The maintainer accepts tasks 4.5 and 4.6 of surface-reminder-schedule-failures as guaranteed by construction. The maintainer accepts task 6.4 of that change and task 4.4 of redirect-shared-temp-to-session-scratch as an eval-credentials gap.
Aaronontheweb
enabled auto-merge (squash)
August 20, 2026 01:13
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
PR #2024 archived 16 completed changes and left two for a maintainer call. The
maintainer approved both for archive. This PR syncs their delta specs and
archives them.
No
.csfile changes.openspec listshows 5 active changes.openspec validate --allpasses (75 items).dotnet slopwatch analyzereports0 issues.
Archived changes
Each change moves to
openspec/changes/archive/2026-08-19-<name>/.surface-reminder-schedule-failuresredirect-shared-temp-to-session-scratchSpec syncs
redirect-shared-temp-to-session-scratchThe OpenSpec CLI applied both delta files:
openspec/specs/session-cwd/spec.md— adds "Session directory is the privateshell scratch location" and modifies "Shell tool failure-path hint for cwd
outside safe spaces".
openspec/specs/tool-approval-gates/spec.md— adds "Platform temporary scopereceives a session-scratch correction", "Intentional platform-temp retry
reaches ordinary approval", and "Parent and subagent scratch corrections are
equivalent".
surface-reminder-schedule-failuresPR #2024 already rewrote "Failure handling and guardrails" in
openspec/specs/netclaw-scheduling/spec.mdfrom thereliable-one-shot-reminder-retrydelta, with corrections against the mergedcode. A programmatic apply would undo those corrections, so this delta was
merged by hand. The added content is:
ConsecutiveFailurescount.
reschedule runs before that occurrence executes.
reconcile restore loop — emits
ReminderScheduleFailedat Warning severity.On the threshold it disables the reminder, emits
ReminderAutoDisabledatCritical severity, and posts a channel notice.
set_remindercreate path and update path return the error to the callerand emit no alert.
Three delta claims are stale against
ReminderManagerActorondevand weredropped:
status: "paused"intasks.json. Themerged code sets
Enabled = falseandTerminalOutcome = Failed. Thecorrected "disable" wording from PR Archive completed OpenSpec changes and sync their specs #2024 stands.
ExecutionTimeoutSecondsconstant onReminderExecutionActor. No such constant exists. The one-hour absolute limitstands.
that PR Archive completed OpenSpec changes and sync their specs #2024 already corrected. The corrected scenarios stand.
Task checkboxes
Task 4.3 of
redirect-shared-temp-to-session-scratchis now ticked. It askedfor native Windows coverage of path and PowerShell behavior. The full
pr_validationsuite ran green on Windows against currentdevon 2026-08-20,from branch
feature/windows-validation-run. That branch is deleted now.These boxes stay unticked, per the maintainer's acceptance:
surface-reminder-schedule-failurestasks 4.5 and 4.6 — parked test tasks.The maintainer accepts them as guaranteed by construction. No reset code was
added at the reschedule sites, and the post-fire path calls the same
ReportScheduleFailureAsyncthat the reconcile tests exercise.surface-reminder-schedule-failurestask 6.4 —./evals/run-evals.shneedseval-target credentials. The maintainer accepts this as an eval-credentials
gap.
redirect-shared-temp-to-session-scratchtask 4.4 — the same eval gate blocksthe last item of this task.
Remaining active changes
memory-core-redesign(4/54)reduce-fresh-session-approval-spam(69/75)streaming-tool-call-execution(33/38)subagent-explicit-model-selection(0/24)memory-relevance-gate(0/14)