ci: bump actions/checkout from 6 to 7 - #1
Merged
Conversation
Bumps [actions/checkout](https://github.com/actions/checkout) from 6 to 7. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v6...v7) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dracic
added a commit
to dracic/bmad-auto
that referenced
this pull request
Jun 24, 2026
…e-on-drift) merge_hooks was idempotent on presence only, so a stale pre-port hook entry (old python3 interpreter, or a claude/Windows entry missing shell: powershell) survived re-init. Detect drift against the canonical _hook_entry and rewrite the managed entry in place; stays idempotent when already canonical and never touches foreign entries. Signature/return contract unchanged (probe.py also relies on it). Refs: _bmad-output/implementation-artifacts/deferred-work.md (deferred-work.md, Adversarial-review deferrals bmad-code-org#1) Claude-Session: https://claude.ai/code/session_019ohWeC44Mto4upsYNNaVh8
dracic
added a commit
to dracic/bmad-auto
that referenced
this pull request
Jun 24, 2026
_copy_library shelled out to GNU cp unconditionally, so Unity Library priming failed on a Windows host without Git-for-Windows coreutils on PATH. Add a shutil.copytree deep-copy fallback guarded by shutil.which("cp") is None, sharing the same volatile/lock/pid stripping and success/failure semantics as the cp path. The cp path is kept where available to preserve CoW reflink benefits.
Refs: _bmad-output/implementation-artifacts/deferred-work.md (deferred-work.md, Windows test-suite greening residuals bmad-code-org#1)
Claude-Session: https://claude.ai/code/session_019ohWeC44Mto4upsYNNaVh8
This was referenced Jul 14, 2026
pbean
added a commit
that referenced
this pull request
Jul 23, 2026
…findings (#276) A second-model adversarial review of PR #277 surfaced 7 valid findings. This lands the fixes for six of them (the 7th, a repo-wide policy-coercion consistency issue, is deferred to a follow-up): - #1 (High) M2 transition now outranks the M1 hash gate. A clean review that round-trips done->in-review->done back to byte-identical launch bytes while omitting its marker no longer false-refuses; the hash gate refuses only when no transition was observed. Safe: the dead-window false positive never records a transition (status never leaves `done`). - #2 (High) The stories-mode folder+id read-back now applies the same launch-snapshot gate, closing the identical false completion it previously bypassed (it accepted on the mtime floor alone). Inert on a dev leg (no snapshot). - #3 (High) The three in-place spec rewriters (append/strip/reset) now write atomically (temp + atomic_replace), so an interrupted/disk-full repair leaves the original spec intact instead of truncated. - #4 (Med) Adds engine-level coverage of the synthesized-REVIEW repair path (including a follow-up review that rewrites the spec) plus regressions for #1/#2/#3/#5/#7 and a _snapshot_verdict truth-table. - #5 (Low) Snapshot/candidate identity is now by resolved filesystem path (_same_spec), not raw string spelling, so a `..`/symlink/case alias can't disable M1/M2. - #7 (Low) A spec ending in a bare `\r` no longer receives an invisible heading (completed to CRLF so the scan's `^` regex sees it). #1 and #2 share a single `_snapshot_verdict` helper so the "M2 outranks M1" rule can't drift between the two paths. All edits sit on `_DevSynthesisMixin`, covering both the tmux and OpenCode adapters.
pbean
added a commit
that referenced
this pull request
Jul 25, 2026
The psmux fix establishes a cross-backend invariant that lived nowhere a backend author reads: `list_window_ids` must emit the same id form `new_window` returns, because `window_alive` is a membership test over it — qualify one side and not the other and every live window reads as instantly dead. #254's implementation notes called this trap #1; the ABC and the authoring guide were both silent on it. Also corrects a claim the fix invalidated: the guide said the parked-window return target is "the one value composed by the backend itself", and psmux now composes two. Notes in the backend why `_qualified_window_id` hand-builds `session:@N` instead of calling `self.target()` (which must stay a by-name token), and that the TUI-side mint/list surfaces deliberately stay bare (#291). Adds the one test the six new ones were missing: #254 is about the `-t` argv, not the return value, so pin that a minted qualified id reaches `pipe-pane -t` verbatim through the override that reimplements that verb.
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
1 participant
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.
Bumps actions/checkout from 6 to 7.
Release notes
Sourced from actions/checkout's releases.
Changelog
Sourced from actions/checkout's changelog.
... (truncated)
Commits
9c091bbupdate error wording (#2467)1044a6dgetting ready for checkout v7 release (#2464)f028218Bump the minor-npm-dependencies group across 1 directory with 3 updates (#2462)d914b26upgrade module to esm and update dependencies (#2463)537c7efBump@actions/coreand@actions/tool-cacheand Remove uuid (#2459)130a169Bump js-yaml from 4.1.0 to 4.2.0 (#2461)7d09575Bump flatted from 3.3.1 to 3.4.2 (#2460)0f9f3aaBump actions/publish-immutable-action (#2458)f9e715ablock checking out fork pr for pull_request_target and workflow_run (#2454)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)