fix(mise): print absolute path when auto-trusting mise.toml (#300)

nsheaps · claude · web-flow · commit 637648e737b5 · 2026-03-24T15:50:52.000-04:00
* fix(mise): resolve and print absolute path when auto-trusting mise.toml Resolve project_dir to an absolute path so trust log messages always show the full path instead of potentially showing '.' or a relative path. Also print the searched directory when no mise.toml is found, making it easier to diagnose why trust was skipped. https://claude.ai/code/session_01EEDHX7L7DiABLZEQCa7EUn * chore: auto-bump plugin versions and update marketplace * feat(scm-utils): rename auto-pr skill to making-great-prs, add /fix-pr command Rename the auto-pr skill to making-great-prs to better reflect its scope as a best-practices guide for PR formatting, not just automation. Add a PR body formatting section (CRITICAL) warning against literal \n escape sequences — the root cause of PR #300's mangled body. This applies to both gh api heredocs and MCP tool string parameters. Add /fix-pr command that evaluates and fixes a PR's title and body to match the skill's formatting standards. https://claude.ai/code/session_01EEDHX7L7DiABLZEQCa7EUn * chore: auto-bump plugin versions and update marketplace --------- Co-authored-by: Claude <noreply@anthropic.com>
diff --git a/.claude-plugin/marketplace.json b/.claude-plugin/marketplace.json
@@ -313,7 +313,7 @@
     {
       "name": "mise",
       "description": "Install and manage mise (tool version manager) in Claude Code sessions. Provides session-start auto-install for web sessions and comprehensive workflow skills.",
-      "version": "0.2.16",
+      "version": "0.2.17",
       "author": {
         "name": "Nathan Heaps"
       },
@@ -454,13 +454,13 @@
     {
       "name": "scm-utils",
       "description": "Source control management utilities for improving interactions with branches and PRs, both locally and in CI environments",
-      "version": "0.1.13",
+      "version": "0.1.14",
       "author": {
         "name": "Nathan Heaps"
       },
       "source": "./plugins/scm-utils",
       "category": "utility",
-      "tags": ["utility", "skill"],
+      "tags": ["utility", "command", "skill"],
       "keywords": ["git", "branch", "pr", "pull-request", "sync", "merge", "scm", "source-control"]
     },
     {
diff --git a/plugins/mise/.claude-plugin/plugin.json b/plugins/mise/.claude-plugin/plugin.json
@@ -1,6 +1,6 @@
 {
   "name": "mise",
-  "version": "0.2.16",
+  "version": "0.2.17",
   "description": "Install and manage mise (tool version manager) in Claude Code sessions. Provides session-start auto-install for web sessions and comprehensive workflow skills.",
   "author": {
     "name": "Nathan Heaps",
diff --git a/plugins/mise/hooks/scripts/install-mise.sh b/plugins/mise/hooks/scripts/install-mise.sh
@@ -108,29 +108,32 @@ do_setup() {
 
   # Auto-trust: trust mise config files in project dir and any git worktrees
   if [ "$auto_trust" = "true" ]; then
-    local project_dir="${CLAUDE_PROJECT_DIR:-.}"
+    local project_dir
+    project_dir="$(cd "${CLAUDE_PROJECT_DIR:-.}" 2>/dev/null && pwd)" || project_dir="${CLAUDE_PROJECT_DIR:-.}"
     local trusted_any=false
 
     # Trust the primary project dir config
     if [ -f "${project_dir}/mise.toml" ]; then
-      hook_log_step "trust-config" "Trusting ${project_dir}/mise.toml"
-      "$mise_bin" trust "${project_dir}/mise.toml" || true
+      local trust_path="${project_dir}/mise.toml"
+      hook_log_step "trust-config" "Trusting ${trust_path}"
+      "$mise_bin" trust "$trust_path" || true
       trusted_any=true
     fi
 
     # Trust mise.toml in any git worktrees
     if command -v git &>/dev/null; then
       while IFS= read -r worktree_dir; do
         if [ "$worktree_dir" != "$project_dir" ] && [ -f "${worktree_dir}/mise.toml" ]; then
-          hook_log "Trusting worktree ${worktree_dir}/mise.toml"
-          "$mise_bin" trust "${worktree_dir}/mise.toml" || true
+          local wt_trust_path="${worktree_dir}/mise.toml"
+          hook_log "Trusting worktree ${wt_trust_path}"
+          "$mise_bin" trust "$wt_trust_path" || true
           trusted_any=true
         fi
       done < <(git -C "$project_dir" worktree list --porcelain 2>/dev/null | grep "^worktree " | sed 's/^worktree //')
     fi
 
     if [ "$trusted_any" = "false" ]; then
-      hook_log "No mise.toml found to trust"
+      hook_log "No mise.toml found to trust (looked in ${project_dir})"
     fi
   fi
 
diff --git a/plugins/scm-utils/.claude-plugin/plugin.json b/plugins/scm-utils/.claude-plugin/plugin.json
@@ -1,6 +1,6 @@
 {
   "name": "scm-utils",
-  "version": "0.1.13",
+  "version": "0.1.14",
   "description": "Source control management utilities for improving interactions with branches and PRs, both locally and in CI environments",
   "author": {
     "name": "Nathan Heaps",
diff --git a/plugins/scm-utils/commands/fix-pr.md b/plugins/scm-utils/commands/fix-pr.md
@@ -0,0 +1,63 @@
+---
+name: fix-pr
+description: Fix or update the PR description for the current branch to follow best practices
+argument-hint: "[optional: PR number or URL]"
+allowed-tools: Bash, Read, Grep, Glob
+---
+
+# Fix PR Description
+
+Review and fix the pull request description for the current branch, ensuring it follows the formatting and content standards from the `making-great-prs` skill.
+
+## Pre-fetched Context
+
+Current branch: !`git branch --show-current 2>/dev/null || echo "(not in a git repo)"`
+
+## Step 1: Identify the PR
+
+Based on **$ARGUMENTS**:
+
+- **No arguments**: Find the open PR for the current branch
+- **PR number or URL provided**: Use the specified PR
+
+## Step 2: Read Current PR State
+
+Fetch the current PR details (title, body, commits, changed files) so you understand:
+
+- What the PR currently says
+- What files were actually changed
+- What the commit history says about the changes
+
+## Step 3: Evaluate and Fix
+
+Check the PR against these standards (from the `making-great-prs` skill):
+
+### Title
+
+- Under 70 characters
+- Starts with a verb (Add, Fix, Update, Refactor, etc.)
+- Matches conventional commit style when applicable
+
+### Body Structure
+
+- Has a `## Summary` section with bullet points describing what changed and why
+- Has a `## Test plan` section with checkbox items for verification
+- Includes a session link if available
+- Uses proper markdown formatting with real newlines (no literal `\n`)
+
+### Body Content
+
+- Summary accurately reflects ALL commits on the branch, not just the latest
+- Test plan items are specific and actionable
+- No placeholder text or template remnants
+
+## Step 4: Update the PR
+
+Update the PR title and/or body with the corrected content. Show the user what changed.
+
+## Usage Examples
+
+```bash
+/fix-pr              # Fix PR for current branch
+/fix-pr 300          # Fix PR #300
+```
diff --git a/plugins/scm-utils/skills/making-great-prs/SKILL.md b/plugins/scm-utils/skills/making-great-prs/SKILL.md
@@ -1,14 +1,15 @@
 ---
-name: auto-pr
+name: making-great-prs
 description: >
-  Automatically create or update a pull request for the current branch.
-  Use after pushing commits to ensure a PR exists and its description is current.
+  Best practices and procedures for creating and maintaining high-quality pull requests.
+  Covers PR creation, body formatting, title conventions, and lifecycle management.
   Triggers on: "create a PR", "update the PR", "open a pull request", "push and PR",
-  or automatically after any push to a feature branch per the auto-pr-management rule.
+  "fix PR", "fix PR body", "PR formatting", or automatically after any push to a
+  feature branch per the auto-pr-management rule.
 allowed-tools: Bash, Read, Grep, Glob
 ---
 
-# Auto PR Management
+# Making Great PRs
 
 Create and maintain pull requests for feature branches using `gh api` with `--hostname github.com` (required for web sessions where the git remote is a local proxy).
 
@@ -104,6 +105,14 @@ gh api repos/nsheaps/ai-mktpl/pulls/<PR_NUMBER> \
 - Start with a verb (Add, Fix, Update, Refactor, etc.)
 - Match conventional commit style when applicable
 
+## PR Body Formatting
+
+**CRITICAL:** PR bodies must contain real newlines, not literal `\n` escape sequences.
+
+- When using `gh api`, use heredocs (`$(cat <<'PREOF' ... PREOF)`) — they preserve newlines naturally
+- When using MCP tools (e.g. `mcp__github__create_pull_request`), pass the body as a multi-line string with actual newlines — MCP tool string parameters support real newlines
+- **Never** construct PR bodies by concatenating strings with `\n` — GitHub renders them as literal text, not line breaks
+
 ## Error Handling
 
 | Error                 | Resolution                                           |

Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "mise",`
`3`		`- "version": "0.2.16",`
	`3`	`+ "version": "0.2.17",`
`4`	`4`	`"description": "Install and manage mise (tool version manager) in Claude Code sessions. Provides session-start auto-install for web sessions and comprehensive workflow skills.",`
`5`	`5`	`"author": {`
`6`	`6`	`"name": "Nathan Heaps",`
Original file line number	Diff line number	Diff line change
`@@ -1,6 +1,6 @@`
`1`	`1`	`{`
`2`	`2`	`"name": "scm-utils",`
`3`		`- "version": "0.1.13",`
	`3`	`+ "version": "0.1.14",`
`4`	`4`	`"description": "Source control management utilities for improving interactions with branches and PRs, both locally and in CI environments",`
`5`	`5`	`"author": {`
`6`	`6`	`"name": "Nathan Heaps",`