Skip to content

chore: surface underlying errors when session history fails to load - #26

Merged
luoxuanzao merged 2 commits into
mainfrom
chore/restore-history-error-detail
Aug 17, 2026
Merged

chore: surface underlying errors when session history fails to load#26
luoxuanzao merged 2 commits into
mainfrom
chore/restore-history-error-detail

Conversation

@luoxuanzao

Copy link
Copy Markdown
Member

Summary

  • The startup restore report only counted failed session history files and discarded the underlying readSDKSession error, so user feedback could never reveal the root cause. The history stage report now appends each failing session id with its error (e.g. Errors: <sessionId>: EPERM: operation not permitted, open '...').
  • This change already proved its worth in the field: a diagnostic build with it let a user's console pinpoint an EPERM caused by endpoint security software blocking content access to ~/.qoder session files — previously invisible behind the bare "failed to load" count.
  • Extended the existing (still unreleased) restore-diagnostics changelog entry to mention the underlying file error; the user-facing Notice is unchanged, details go to the developer console only.

Verification

  • npm run typecheck
  • npm run lint
  • npm run test (3323 passed; new assertion covers the surfaced error detail)
  • npm run build
  • npm run release:check
  • npm run audit:prod
  • Tested affected qodercli behavior against a real CLI when applicable (a diagnostic build of this change was installed by an affected user; the console then showed the underlying EPERM error with the failing session path, confirming the enhancement works end to end)

Safety

  • No credentials, private vault content, internal URLs, or personal paths are included
  • User-visible changes are documented in CHANGELOG.md

luoxuanzao and others added 2 commits August 17, 2026 15:21
The restore report only counted failed session files, discarding the underlying readSDKSession error, so user feedback could not reveal the root cause. Append each failing session id and its error to the reported detail.

Co-authored-by: QoderAI (Qwen 3.8 Max) <qoder_ai@qoder.com>
…elog entry

Co-authored-by: QoderAI (Qwen 3.8 Max) <qoder_ai@qoder.com>
@luoxuanzao
luoxuanzao merged commit 7b4e20a into main Aug 17, 2026
10 checks passed
@luoxuanzao
luoxuanzao deleted the chore/restore-history-error-detail branch August 17, 2026 11:41
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.

1 participant