Skip to content

fix(cli): Preserve conversation config clears - #1160

Merged
JeanMertz merged 1 commit into
mainfrom
config-clear-by-path-2
Sep 16, 2026
Merged

JeanMertz merged 1 commit into
mainfrom
config-clear-by-path-2

Conversation

@JeanMertz

Copy link
Copy Markdown
Collaborator

Fields cleared with jp query --cfg assistant.name:=null stay cleared on subsequent invocations, even when workspace config supplies a value. The next query no longer records a delta restoring that value.

Carry active conversation clears through base-layer filling before applying --cfg overrides. Explicit values still win, later sets retire clears, and resets discard earlier clears. Existing stored clears work without a data migration.

Fixes: T-0gqh7z7

Fields cleared with `jp query --cfg assistant.name:=null` stay cleared
on subsequent invocations, even when workspace config supplies a value.
The next query no longer records a delta restoring that value.

Carry active conversation clears through base-layer filling before
applying `--cfg` overrides. Explicit values still win, later sets retire
clears, and resets discard earlier clears. Existing stored clears work
without a data migration.

Fixes: T-0gqh7z7
Signed-off-by: Jean Mertz <git@jeanmertz.com>
@JeanMertz
JeanMertz force-pushed the config-clear-by-path-2 branch from 36a48ab to c471624 Compare September 10, 2026 21:24
@JeanMertz
JeanMertz merged commit 4582307 into main Sep 16, 2026
21 checks passed
@JeanMertz
JeanMertz deleted the config-clear-by-path-2 branch September 16, 2026 12:28
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