Skip to content

Commit d6855b6

Browse files
gtrrz-victorclaude
andcommitted
refactor(test): drop issue-number refs and redundant integration test (review)
The unit test in lifecycle_test.go already pins the stripped prompt end-to-end through ParseHookEvent, so the integration file duplicated coverage. Issue references removed from code comments per review. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Entire-Checkpoint: 01KXJNYKYC9AK1B7PC6ZAW6AZP
1 parent 325ea8b commit d6855b6

3 files changed

Lines changed: 4 additions & 46 deletions

File tree

cmd/entire/cli/agent/claudecode/lifecycle.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ func (c *ClaudeCodeAgent) parseTurnStart(stdin io.Reader) (*agent.Event, error)
140140
SessionRef: raw.TranscriptPath,
141141
// Strip IDE-injected context (e.g. <ide_opened_file> from the VS Code
142142
// extension) so the session/checkpoint title and prompt show what the
143-
// user actually typed, not the injected block (#1423).
143+
// user actually typed, not the injected block.
144144
Prompt: textutil.StripIDEContextTags(raw.Prompt),
145145
Timestamp: time.Now(),
146146
}, nil

cmd/entire/cli/agent/claudecode/lifecycle_test.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -99,9 +99,9 @@ func TestParseHookEvent_TurnStart(t *testing.T) {
9999
}
100100
}
101101

102-
// Regression for #1423: the VS Code extension prepends an <ide_opened_file>
103-
// context block to the prompt; it must be stripped so the session/checkpoint
104-
// title and prompt show only what the user typed.
102+
// The VS Code extension prepends an <ide_opened_file> context block to the
103+
// prompt; it must be stripped so the session/checkpoint title and prompt show
104+
// only what the user typed.
105105
func TestParseHookEvent_TurnStart_StripsIDEContextTags(t *testing.T) {
106106
t.Parallel()
107107

cmd/entire/cli/integration_test/issue_1423_ide_tag_e2e_test.go

Lines changed: 0 additions & 42 deletions
This file was deleted.

0 commit comments

Comments
 (0)