[PR #24] [MERGED] include original session file path in handoff output #22

New issue

Closed

opened 2026-03-04 01:39:18 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 01:39:18 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/yigitkonur/cli-continues/pull/24
Author: @yigitkonur
Created: 3/2/2026
Status: ✅ Merged
Merged: 3/2/2026
Merged by: @yigitkonur

Base: main ← Head: feat/session-file-path-in-handoff

📝 Commits (1)

06ae8ba include original session file path in handoff output

📊 Changes

3 files changed (+39 additions, -2 deletions)

View changed files

📝 .gitignore (+9 -0)
📝 src/utils/markdown.ts (+26 -0)
📝 src/utils/resume.ts (+4 -2)

📄 Description

hey! so i noticed when you do a cross-tool handoff, the markdown output doesn't actually tell you where the original session file lives on disk. like you get the session id and working directory, but if you ever want to go back and check the raw session data or debug something, you're kinda stuck guessing the path.

this adds the full file path in a few places:

session overview table — new Session File row right after the working directory, so it's easy to spot
session origin section — added at the bottom of the handoff markdown, with the source tool name, file path, session id, and project dir. also has a little note about inspecting the raw data
inline resume prompt — now mentions the original session path in the intro line
reference resume prompt — added an Original session file row to the quick-reference table

all of these are conditional on session.originalPath being set, which every parser already populates, so nothing breaks. no new dependencies, no schema changes.

files changed

src/utils/markdown.ts — overview table row + session origin section
src/utils/resume.ts — inline prompt + reference prompt updates

tested locally, all existing tests pass (the zod-related failures are pre-existing and unrelated).

Review all of them with eye of John Carmack-like simplicity with elegeance approach and apply the one only if required

Greptile Summary

Adds originalPath (session file path) to handoff outputs. The overview table addition (line 125) and reference prompt update are good — they surface useful debugging info. However, the "Session Origin" section (lines 240-252) duplicates data already in the overview table (session ID, cwd, file path). Cut the redundant section, keep the table row.

What's good:

Session file now visible in overview table
Reference prompt correctly includes the path (it doesn't embed the full markdown)
All changes are conditional on originalPath being set

What's unnecessary:

The "Session Origin" section at the end repeats information from the top

Confidence Score: 4/5

Safe to merge after removing the redundant "Session Origin" section
Core functionality is solid — adds session file path to the right places. One issue: the "Session Origin" section duplicates information already in the overview table. No bugs, no security concerns, just unnecessary padding that should be cut.
Pay attention to src/utils/markdown.ts — remove the redundant "Session Origin" section (lines 240-252)

Important Files Changed

Filename	Overview
src/utils/markdown.ts	Adds session file path to overview table (good) but also adds redundant "Session Origin" section that duplicates existing information
src/utils/resume.ts	Adds session file path to inline and reference prompts - minor redundancy in inline mode but reference mode addition is useful

_{Last reviewed commit: 8969ddf}

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/yigitkonur/cli-continues/pull/24 **Author:** [@yigitkonur](https://github.com/yigitkonur) **Created:** 3/2/2026 **Status:** ✅ Merged **Merged:** 3/2/2026 **Merged by:** [@yigitkonur](https://github.com/yigitkonur) **Base:** `main` ← **Head:** `feat/session-file-path-in-handoff` --- ### 📝 Commits (1) - [`06ae8ba`](https://github.com/yigitkonur/cli-continues/commit/06ae8ba85c47c47e4a1fc53bd1eeaff4fdafd991) include original session file path in handoff output ### 📊 Changes **3 files changed** (+39 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `.gitignore` (+9 -0) 📝 `src/utils/markdown.ts` (+26 -0) 📝 `src/utils/resume.ts` (+4 -2) </details> ### 📄 Description hey! so i noticed when you do a cross-tool handoff, the markdown output doesn't actually tell you where the original session file lives on disk. like you get the session id and working directory, but if you ever want to go back and check the raw session data or debug something, you're kinda stuck guessing the path. this adds the full file path in a few places: - **session overview table** — new `Session File` row right after the working directory, so it's easy to spot - **session origin section** — added at the bottom of the handoff markdown, with the source tool name, file path, session id, and project dir. also has a little note about inspecting the raw data - **inline resume prompt** — now mentions the original session path in the intro line - **reference resume prompt** — added an `Original session file` row to the quick-reference table all of these are conditional on `session.originalPath` being set, which every parser already populates, so nothing breaks. no new dependencies, no schema changes. ### files changed - `src/utils/markdown.ts` — overview table row + session origin section - `src/utils/resume.ts` — inline prompt + reference prompt updates tested locally, all existing tests pass (the zod-related failures are pre-existing and unrelated).  --- <a href="https://app.devin.ai/review/yigitkonur/cli-continues/pull/24" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a>   # Review all of them with eye of John Carmack-like simplicity with elegeance approach and apply the one only if required <h3>Greptile Summary</h3> Adds `originalPath` (session file path) to handoff outputs. The overview table addition (line 125) and reference prompt update are good — they surface useful debugging info. However, the "Session Origin" section (lines 240-252) duplicates data already in the overview table (session ID, cwd, file path). Cut the redundant section, keep the table row. **What's good:** - Session file now visible in overview table - Reference prompt correctly includes the path (it doesn't embed the full markdown) - All changes are conditional on `originalPath` being set **What's unnecessary:** - The "Session Origin" section at the end repeats information from the top <details><summary><h3>Confidence Score: 4/5</h3></summary> - Safe to merge after removing the redundant "Session Origin" section - Core functionality is solid — adds session file path to the right places. One issue: the "Session Origin" section duplicates information already in the overview table. No bugs, no security concerns, just unnecessary padding that should be cut. - Pay attention to `src/utils/markdown.ts` — remove the redundant "Session Origin" section (lines 240-252) </details> <details open><summary><h3>Important Files Changed</h3></summary> | Filename | Overview | |----------|----------| | src/utils/markdown.ts | Adds session file path to overview table (good) but also adds redundant "Session Origin" section that duplicates existing information | | src/utils/resume.ts | Adds session file path to inline and reference prompts - minor redundancy in inline mode but reference mode addition is useful | </details> </details> <sub>Last reviewed commit: 8969ddf</sub>   --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>