Skip to content

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

Open
JeanMertz wants to merge 2 commits into
mainfrom
config-clear-by-path-2
Open

JeanMertz wants to merge 2 commits 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>
Signed-off-by: Jean Mertz <git@jeanmertz.com>
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