Skip to content

Commit 2956a80

Browse files
committed
fix(ci): cross-platform export-memory test + restore MSRV toolchain pin
- export_memory_dry_run test: assert filename + 'memory' separately instead of the 'memory/<file>' literal — Windows prints a backslash separator. - ci: msrv job pins dtolnay/rust-toolchain@1.88 again (dependabot #17 bumped it to @1.100, which installs a non-existent rust 1.100.0 → 404).
1 parent ec28ad7 commit 2956a80

2 files changed

Lines changed: 7 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,10 @@ jobs:
5050
- uses: actions/checkout@v4
5151

5252
- name: Install Rust 1.88
53-
uses: dtolnay/rust-toolchain@1.100
53+
# The action tag IS the toolchain version installed; @1.88 pins MSRV.
54+
# (Dependabot bumped this to @1.100 in #17, which tries to install a
55+
# non-existent rust 1.100.0 — revert to the real MSRV.)
56+
uses: dtolnay/rust-toolchain@1.88
5457

5558
- name: Cache cargo
5659
uses: Swatinem/rust-cache@v2

crates/tj-cli/tests/cli.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4527,7 +4527,9 @@ fn export_memory_dry_run_prints_path_and_content_no_write() {
45274527
.args(["export-memory", "--task", &task_id, "--dry-run"])
45284528
.assert()
45294529
.success()
4530-
.stdout(contains(format!("memory/tj-{task_id}-ship-x.md")))
4530+
// Separator-agnostic: Windows prints `memory\tj-...` not `memory/tj-...`.
4531+
.stdout(contains(format!("tj-{task_id}-ship-x.md")))
4532+
.stdout(contains("memory"))
45314533
.stdout(contains("name: ship-x"));
45324534

45334535
// Nothing written under the sandboxed Claude config dir.

0 commit comments

Comments
 (0)