-
-
Notifications
You must be signed in to change notification settings - Fork 79.1k
[Bug]: encoded trajectory export request can erase an explicit session key #83282
Copy link
Copy link
Closed
Closed
Copy link
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:session-stateSession, memory, transcript, context, or agent state can drift or corrupt.Session, memory, transcript, context, or agent state can drift or corrupt.
Metadata
Metadata
Assignees
Labels
P2Normal backlog priority with limited blast radius.Normal backlog priority with limited blast radius.clawsweeper:linked-pr-openClawSweeper found an open linked pull request for this issue.ClawSweeper found an open linked pull request for this issue.clawsweeper:no-new-fix-prClawSweeper does not recommend queueing a new automated fix PR for this issue.ClawSweeper does not recommend queueing a new automated fix PR for this issue.clawsweeper:source-reproClawSweeper found a high-confidence source-level issue reproduction.ClawSweeper found a high-confidence source-level issue reproduction.impact:session-stateSession, memory, transcript, context, or agent state can drift or corrupt.Session, memory, transcript, context, or agent state can drift or corrupt.
Type
Fields
Give feedbackNo fields configured for issues without a type.
Summary
A local code audit found that merging an encoded export-trajectory request can overwrite an explicitly supplied session key with an empty placeholder when the encoded request omits
sessionKey.Affected area
src/commands/export-trajectory.tsdecodeExportTrajectoryRequestresolveExportTrajectoryOptionsRepro
Call the trajectory export command with a direct
sessionKeyand a base64url request that only supplies another option such asoutput. The resolved options can lose the direct session key and fail with the--session-key is requiredpath.Expected
Decoded request values should not overwrite already-defined CLI options with empty placeholders.
Suggested fix
Only include
sessionKeyin decoded partial options when the encoded request contains a non-empty string, or merge decoded values without replacing defined direct options with empty values.Source: local code audit.