Skip to content

feat: add recovery summary to session history export - #46

Merged
johnnyhuy merged 1 commit into
mainfrom
agent/issue-45-export-recovery-summary
Nov 10, 2026
Merged

johnnyhuy merged 1 commit into
mainfrom
agent/issue-45-export-recovery-summary

Conversation

@johnnyhuy

@johnnyhuy johnnyhuy commented Nov 10, 2026 •

Copy link
Copy Markdown
Contributor

Exported session Markdown now includes a recovery summary for non-empty selections. It counts messages from you and Pi, counts completed, failed and running tool calls, and lists distinct tool names in first-use order. Empty exports keep their existing shape.

Part of #4
Closes #45

Verification

  • bun install --frozen-lockfile passed with the pinned mise toolchain, Node 22.23.0 and Bun 1.3.14.
  • mise run build passed, including bun run lint, bun run typecheck, bun run test:unit (84 tests), bun run test:integration (46 tests), main/preload compilation and Vite bundling.
  • The focused export suite passed all 10 tests. Its output names summarises tool outcomes and distinct tools in first-use order, handles an empty selection and a blank title, and writes the full conversation with context and tool input.
  • The earlier localStorage integration failures did not reproduce with the pinned Node version. The earlier mise sandbox restriction was cleared for this verification run.

bun run test:e2e passed all 43 tests across five files (151.65 seconds). The previous loopback sandbox block was cleared with the required desktop/loopback access.

Proof

Capture methods (a), (c) and (d) succeeded. E2E output is saved locally at .tmp/proof/pr-46-e2e.log. The inspected session-history screenshot at .tmp/proof/pr-46-history.png is supporting evidence of the existing Export surface, not a picture of the exported summary. The terminal capture below comes from the real sessionExportMarkdown function at PR head 3491ef2125d2ca3866cf6483740ada61f909a225. Local scratch captures are .tmp/proof/pr-46-build.log and .tmp/proof/pr-46-export.txt; they are not committed. The visible change is exported text, with no new in-app surface.

# Move context to a new folder

Exported from pidex at 2026-11-10T00:00:00.000Z.

It ran in `/repo/app`.

It used `anthropic/claude-test`.

## Summary

- 2 messages: 1 from you, 1 from Pi.
- 2 tool calls: 1 done, 1 failed, 0 running.
- Tools used: bash, read.

Gaps

This slice summarises exported text. In-app recovery summaries, folder-to-folder archive relocation and a migration wizard remain outside this slice. The Vite build reports its existing large-chunk warning, and e2e logs SVG property warnings. Neither failed a gate. All requested verification gates passed; no environment gaps remain for this run.

@johnnyhuy
johnnyhuy merged commit f122358 into main Nov 10, 2026
15 checks passed
@johnnyhuy
johnnyhuy deleted the agent/issue-45-export-recovery-summary branch November 10, 2026 15:32
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.

Add a recovery summary to the session history export

1 participant