Greptile Summary

This PR fixes a silent data-loss path in the QMD session export by extending listSessionFilesForAgent() to include archived transcript files (.jsonl.reset.<timestamp> and .jsonl.deleted.<timestamp>) alongside live .jsonl files. The change is intentionally minimal: a single compiled regex replaces the .endsWith(".jsonl") check, readdir still operates non-recursively so nested sessions/archive/ entries remain out of scope, and the existing call sites are unaffected.

Key observations:

• The regex /\.jsonl(?:\.(?:reset|deleted)\..+)?$/ is correct and covers all described cases — it properly rejects .jsonl.lock, .jsonl.reset (no trailing segment), and .jsonl.reset. (empty trailing segment), while matching the two new archive patterns.
• No risk of ReDoS — the regex has no nested quantifiers or catastrophic backtracking paths.
• The new test uses vi.stubEnv("OPENCLAW_HOME", tmpDir) correctly and verifies all four boundary cases (live, reset, deleted, ignored, nested directory).
• One downstream consideration worth being aware of: sessionPathForFile() (not changed here) will produce keys like sessions/session.jsonl.reset.2026-03-12T08-50-29.182Z for archived files. Callers that use these as output document names in the QMD export may want to normalise or sanitise them — but that is explicitly noted as out of scope for this PR.

Confidence Score: 5/5

• This PR is safe to merge — the change is minimal, well-tested, and non-breaking.
• The modification is a single-line filter replacement backed by a carefully crafted, non-catastrophic regex. Live session handling is unchanged, archived file matching is verified by a dedicated test, and no downstream API surfaces are altered. No logical errors, security issues, or regressions were found.
• No files require special attention.

_{Last reviewed commit: 4d4ebb4}

Follow-up after opening this PR: I found related existing threads that I should have linked up front.

Related:

• Issue [Bug]: QMD session exporter ignores reset/deleted session transcripts (.jsonl.reset.*, .jsonl.deleted.*) #30220 — QMD session exporter ignores .jsonl.reset.* / .jsonl.deleted.*
• PR fix(qmd): include archived session transcripts in file listing #30273 — very similar fix for archived session transcript listing
• PR fix(memory): index reset session transcripts #20183 — broader alternative with opt-in reset archive indexing/config plumbing

So for maintainers: this PR should be read as a minimal alternative to the broader variants, not as the first report/fix for the problem.

What is intentionally different here:

• minimal surface area
• no config/schema changes
• includes both .jsonl.reset.* and .jsonl.deleted.*
• only covers files in the agent sessions root
• deliberately leaves sessions/archive/ out of scope for a separate product decision

Also, the follow-up push fixes the Windows CI failure from the first version of the new test by making the test state-dir-aware (OPENCLAW_STATE_DIR + OPENCLAW_HOME).

If maintainers prefer consolidating on #30273 or #20183 instead of keeping this smaller variant, I'm happy to close this PR.

Thanks for the fix here.

Closing this as superseded by #57446, which carries the archived-session export fix onto the current packages/memory-host-sdk path on top of latest main.

I kept the same small scope: include reset/deleted transcript history without widening into unrelated archive traversal behavior.

Thanks for picking this up and porting it to the current code path, @vincentkoc! Glad the fix made it into #57446 — happy to see the archived transcript coverage land properly on top of the new memory-host-sdk structure. Appreciate you taking the time to review and carry it forward. 🙌