Skip to content

chore: Refine tool configs, personas, and skills#511

Merged
JeanMertz merged 2 commits intomainfrom
prr80
Apr 4, 2026
Merged

chore: Refine tool configs, personas, and skills#511
JeanMertz merged 2 commits intomainfrom
prr80

Conversation

@JeanMertz
Copy link
Copy Markdown
Collaborator

Several JP configuration files are improved in this pass.

The git_diff tool drops the "all" status option — only "staged" and "unstaged" remain. A description is added noting that per-file diffs are truncated at 50 lines, with guidance to re-run using paths for full output. The git-reading skill prompt is updated to match.

fs_modify_file switches from asking the assistant to review proposed changes to auto-approving them (apply_changes.answer = true). The rejection error message is also reworded to be more actionable.

The stager persona gains a proper [conversation.tools] section, replacing the inline tools.'*' setting. All tools now default to run = "ask", with git_stage_patch staging confirmations auto-approved. The dev persona adds auto-approval for fs_create_file file overwrites.

Tool style settings for cargo_expand, git_stage_patch, github_code_search, and web_fetch are moved into dedicated [style] subsections, making the structure more consistent. results_file_link and parameters style options are added where appropriate.

The edit-files skill prompt adds an explicit reminder to use fs_move_file instead of delete-and-recreate for file moves.

Several JP configuration files are improved in this pass.

The `git_diff` tool drops the `"all"` status option — only `"staged"`
and `"unstaged"` remain. A description is added noting that per-file
diffs are truncated at 50 lines, with guidance to re-run using `paths`
for full output. The `git-reading` skill prompt is updated to match.

`fs_modify_file` switches from asking the assistant to review proposed
changes to auto-approving them (`apply_changes.answer = true`). The
rejection error message is also reworded to be more actionable.

The `stager` persona gains a proper `[conversation.tools]` section,
replacing the inline `tools.'*'` setting. All tools now default to `run
= "ask"`, with `git_stage_patch` staging confirmations auto-approved.
The `dev` persona adds auto-approval for `fs_create_file` file
overwrites.

Tool style settings for `cargo_expand`, `git_stage_patch`,
`github_code_search`, and `web_fetch` are moved into dedicated `[style]`
subsections, making the structure more consistent. `results_file_link`
and `parameters` style options are added where appropriate.

The `edit-files` skill prompt adds an explicit reminder to use
`fs_move_file` instead of delete-and-recreate for file moves.

Signed-off-by: Jean Mertz <git@jeanmertz.com>
Signed-off-by: Jean Mertz <git@jeanmertz.com>
@JeanMertz JeanMertz merged commit c6bae63 into main Apr 4, 2026
13 checks passed
@JeanMertz JeanMertz deleted the prr80 branch April 4, 2026 07:08
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