chore: Add Claude Code commands for common development and git workflows (#725)

Adds Claude commands for common development and git workflows: `/check`,
`/commit-push`, `/pr`, and `/commit-push-pr`

How and whether these commands work may depend on your local Claude Code
setup, especially if `sandbox.enabled` is `true`. Claude can help debug
any issues you're experiencing on your system. `/check` should always
work.

A handful of package scripts are made quieter on success in order to
preserve our precious tokens.

Note that these are [custom slash
commands](https://code.claude.com/docs/en/slash-commands#custom-slash-commands),
which is not the same as
[skills](https://code.claude.com/docs/en/skills).


<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Adds Claude Code automation and streamlines package scripts for
lower-noise CI/local runs.
> 
> - **New commands**:
`/.claude/commands/{check,commit-push,pr,commit-push-pr}.md` for
lint/build/test, commit/push, and PR creation workflows
> - **Script noise reduction**: switch `depcheck` to `--quiet`, add
`prettier --log-level error`, use `vitest --reporter dot` for
`test:dev`, and silence root `yarn dedupe` stdout
> - **Repo constraints updated**: `yarn.config.cjs` expectations aligned
with the new `lint:*` and `test:*` script values across all packages
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
0814ffa. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

---------

Co-authored-by: Claude <[email protected]>
Co-authored-by: claude[bot] <41898282+claude[bot]@users.noreply.github.com>
Co-authored-by: Erik Marks <[email protected]>

Author: 3 people

.claude/commands/check.md

@@ -0,0 +1,28 @@
+---
+name: check
+description: Lints, builds, and tests the code.
+argument-hint: [package-name]
+allowed-tools:
+  - Bash(yarn lint*)
+  - Bash(yarn build*)
+  - Bash(yarn test*)
+  - Bash(yarn workspace *)
+model: claude-haiku-4-5
+---
+
+Run the following commands to check the current state of the code:
+
+If `$ARGUMENTS` is provided (e.g. `@metamask/ocap-kernel`), run the commands in
+that workspace using `yarn workspace $ARGUMENTS`:
+
+1. `yarn workspace $ARGUMENTS lint:fix`
+2. `yarn workspace $ARGUMENTS build`
+3. `yarn workspace $ARGUMENTS test:dev`
+
+If `$ARGUMENTS` is empty, run the commands at the monorepo root:
+
+1. `yarn lint:fix`
+2. `yarn build`
+3. `yarn test:dev`
+
+Return the results of the commands.

.claude/commands/commit-push-pr.md

@@ -0,0 +1,10 @@
+---
+name: commit-push-pr
+description: Optionally checks, then commits and pushes code to the remote repository, and creates a pull request.
+argument-hint: check | force
+allowed-tools:
+  - Skill
+model: claude-haiku-4-5
+---
+
+Run `/commit-push $ARGUMENTS` then `/pr`.

.claude/commands/commit-push.md

@@ -0,0 +1,56 @@
+---
+name: commit-push
+description: Optionally checks, then commits and pushes code to the remote repository.
+argument-hint: check | force
+allowed-tools:
+  - Bash(git branch*)
+  - Bash(git checkout*)
+  - Bash(git add*)
+  - Bash(git status*)
+  - Bash(git commit*)
+  - Bash(git push*)
+  - Bash(git diff*)
+  - Bash(git log*)
+  - Skill
+model: claude-haiku-4-5
+---
+
+Arguments: $ARGUMENTS
+
+If the argument is "force", skip the check step. Otherwise (default), run the `/check` command first to lint, build, and test the code. If any of the checks fail, stop and report the errors.
+
+Once ready, commit and push the code by following these steps:
+
+1. Run these bash commands in parallel to understand the current state:
+
+   - `git status` to see all untracked files
+   - `git diff HEAD` to see both staged and unstaged changes
+   - `git log --oneline -10` to see recent commit messages for style consistency
+
+2. If you are on the `main` branch, create a new feature branch using `git branch` and switch to it.
+
+3. Analyze all changes and draft a commit message:
+
+   - Summarize the nature of the changes (new feature, enhancement, bug fix, refactoring, test, docs, etc.)
+   - Use the conventional commit format: `type(scope): description`
+   - Keep the first line under 72 characters
+   - Do not commit files that likely contain secrets (.env, credentials.json, etc.)
+
+4. Stage and commit the changes:
+
+   - Add relevant files using `git add`
+   - Create the commit with a message ending with:
+     ```
+     Co-Authored-By: Claude <[email protected]>
+     ```
+   - Use a HEREDOC for the commit message to ensure proper formatting
+
+5. Push to the remote repository:
+
+   - Run `git push` to push the commit
+   - If the branch has no upstream, use `git push -u origin <branch-name>`
+
+6. Report the results including:
+   - The commit hash
+   - The commit message
+   - The push status

.claude/commands/pr.md

@@ -0,0 +1,20 @@
+---
+name: pr
+description: Creates a pull request for the current branch.
+allowed-tools:
+  - Bash(git status*)
+  - Bash(git log*)
+  - Bash(git show*)
+  - Bash(git diff*)
+  - Bash(gh pr create*)
+model: claude-haiku-4-5
+---
+
+1. Run `git status`. If any of the following conditions apply, stop and report the errors:
+   - There are unstaged changes
+   - There are untracked files
+   - The current branch is the default branch (`main`)
+2. Run `git log main..HEAD --oneline` to see the commit history.
+3. Run `git diff` and/or `git show` as necessary to understand the changes.
+4. Run `gh pr create` to create a pull request.
+5. Return the results of the commands.

package.json

@@ -20,18 +20,18 @@
     "clean": "rimraf --glob './*.tsbuildinfo' ./.eslintcache ./coverage ./.turbo && yarn workspaces foreach --all --parallel --interlaced --verbose run clean",
     "create-package": "tsx packages/create-package/src/index.ts",
     "lint": "yarn constraints && yarn lint:eslint && yarn lint:misc --check && yarn lint:dependencies",
-    "lint:dependencies": "yarn dedupe --check && yarn depcheck && yarn workspaces foreach --all --parallel --verbose run lint:dependencies",
-    "lint:dependencies:fix": "yarn dedupe && yarn depcheck && yarn workspaces foreach --all --parallel --verbose run lint:dependencies",
+    "lint:dependencies": "yarn dedupe --check >/dev/null && yarn depcheck --quiet && yarn workspaces foreach --all --parallel --verbose run lint:dependencies",
+    "lint:dependencies:fix": "yarn dedupe >/dev/null && yarn depcheck --quiet && yarn workspaces foreach --all --parallel --verbose run lint:dependencies",
     "lint:eslint": "yarn eslint . --cache",
     "lint:fix": "yarn constraints --fix && yarn lint:eslint --fix && yarn lint:misc --write && yarn lint:dependencies:fix",
-    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html'  '**/*.yml' '!**/CHANGELOG.old.md' '!.yarnrc.yml' '!CLAUDE.md' '!merged-packages/**' --ignore-path .gitignore",
+    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html'  '**/*.yml' '!**/CHANGELOG.old.md' '!.yarnrc.yml' '!CLAUDE.md' '!merged-packages/**' --ignore-path .gitignore  --log-level error",
     "postinstall": "simple-git-hooks && yarn rebuild:native",
     "prepack": "./scripts/prepack.sh",
     "pretest": "bash scripts/reset-coverage-thresholds.sh",
     "rebuild:native": "./scripts/rebuild-native.sh",
     "test": "yarn pretest && vitest run",
     "test:ci": "vitest run --coverage false",
-    "test:dev": "yarn test --mode development",
+    "test:dev": "yarn test --mode development --reporter dot",
     "test:e2e": "yarn workspaces foreach --all run test:e2e",
     "test:e2e:ci": "yarn workspaces foreach --all run test:e2e:ci",
     "test:verbose": "yarn test --reporter verbose",

packages/brow-2-brow/package.json

@@ -15,13 +15,13 @@
     "changelog:validate": "../../scripts/validate-changelog.sh @ocap/brow-2-brow",
     "clean": "rimraf --glob './*.tsbuildinfo' ./.eslintcache ./coverage ./dist ./.turbo",
     "lint": "yarn lint:eslint && yarn lint:misc --check && yarn constraints && yarn lint:dependencies",
-    "lint:dependencies": "depcheck",
+    "lint:dependencies": "depcheck --quiet",
     "lint:eslint": "eslint . --cache",
     "lint:fix": "yarn lint:eslint --fix && yarn lint:misc --write && yarn constraints --fix && yarn lint:dependencies",
-    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore",
+    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore --log-level error",
     "test": "vitest run --config vitest.config.ts",
     "test:clean": "yarn test --no-cache --coverage.clean",
-    "test:dev": "yarn test --mode development",
+    "test:dev": "yarn test --mode development --reporter dot",
     "test:verbose": "yarn test --reporter verbose",
     "test:watch": "vitest --config vitest.config.ts",
     "start": "npm run build:dev && esbuild src/index.js --serve --sourcemap --bundle --outdir=dist --servedir=dist --external:@types/web",

packages/cli/package.json

@@ -20,14 +20,14 @@
     "changelog:validate": "../../scripts/validate-changelog.sh @ocap/cli",
     "clean": "rimraf --glob './*.tsbuildinfo' ./.eslintcache ./coverage ./dist ./.turbo",
     "lint": "yarn lint:eslint && yarn lint:misc --check && yarn constraints && yarn lint:dependencies",
-    "lint:dependencies": "depcheck",
+    "lint:dependencies": "depcheck --quiet",
     "lint:eslint": "eslint . --cache",
     "lint:fix": "yarn lint:eslint --fix && yarn lint:misc --write && yarn constraints --fix && yarn lint:dependencies",
-    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore",
+    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore --log-level error",
     "publish:preview": "yarn npm publish --tag preview",
     "test": "vitest run --config vitest.config.ts",
     "test:clean": "yarn test --no-cache --coverage.clean",
-    "test:dev": "yarn test --mode development",
+    "test:dev": "yarn test --mode development --reporter dot",
     "test:verbose": "yarn test --reporter verbose",
     "test:watch": "vitest --config vitest.config.ts",
     "test:integration": "vitest run --config vitest.integration.config.ts"

packages/create-package/package.json

@@ -33,14 +33,14 @@
     "changelog:validate": "../../scripts/validate-changelog.sh @ocap/create-package",
     "clean": "rimraf --glob './*.tsbuildinfo' ./.eslintcache ./coverage ./dist ./.turbo",
     "lint": "yarn lint:eslint && yarn lint:misc --check && yarn constraints && yarn lint:dependencies",
-    "lint:dependencies": "depcheck",
+    "lint:dependencies": "depcheck --quiet",
     "lint:eslint": "eslint . --cache",
     "lint:fix": "yarn lint:eslint --fix && yarn lint:misc --write && yarn constraints --fix && yarn lint:dependencies",
-    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore",
+    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore --log-level error",
     "publish:preview": "yarn npm publish --tag preview",
     "test": "vitest run --config vitest.config.ts",
     "test:clean": "yarn test --no-cache --coverage.clean",
-    "test:dev": "yarn test --mode development",
+    "test:dev": "yarn test --mode development --reporter dot",
     "test:verbose": "yarn test --reporter verbose",
     "test:watch": "vitest --config vitest.config.ts"
   },

packages/extension/package.json

@@ -23,17 +23,17 @@
     "changelog:validate": "../../scripts/validate-changelog.sh @ocap/extension",
     "clean": "rimraf --glob './*.tsbuildinfo' ./.eslintcache ./coverage ./dist ./.turbo",
     "lint": "yarn lint:eslint && yarn lint:misc --check && yarn constraints && yarn lint:dependencies",
-    "lint:dependencies": "depcheck",
+    "lint:dependencies": "depcheck --quiet",
     "lint:eslint": "eslint . --cache",
     "lint:fix": "yarn lint:eslint --fix && yarn lint:misc --write && yarn constraints --fix && yarn lint:dependencies",
-    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore",
+    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore --log-level error",
     "publish:preview": "yarn npm publish --tag preview",
     "postinstall": "yarn playwright install chromium",
     "start": "./scripts/start.sh",
     "test": "vitest run --config vitest.config.ts",
     "test:build": "tsx ./test/build/build-tests.ts",
     "test:clean": "yarn test --no-cache --coverage.clean",
-    "test:dev": "yarn test --mode development",
+    "test:dev": "yarn test --mode development --reporter dot",
     "test:verbose": "yarn test --reporter verbose",
     "test:watch": "vitest --config vitest.config.ts",
     "test:e2e": "yarn playwright test",

packages/kernel-agents/package.json

@@ -54,15 +54,15 @@
     "changelog:validate": "../../scripts/validate-changelog.sh @ocap/kernel-agents",
     "clean": "rimraf --glob './*.tsbuildinfo' ./.eslintcache ./coverage ./dist",
     "lint": "yarn lint:eslint && yarn lint:misc --check && yarn constraints && yarn lint:dependencies",
-    "lint:dependencies": "depcheck",
+    "lint:dependencies": "depcheck --quiet",
     "lint:eslint": "eslint . --cache",
     "lint:fix": "yarn lint:eslint --fix && yarn lint:misc --write && yarn constraints --fix && yarn lint:dependencies",
-    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore",
+    "lint:misc": "prettier --no-error-on-unmatched-pattern '**/*.json' '**/*.md' '**/*.html' '!**/CHANGELOG.old.md' '**/*.yml' '!.yarnrc.yml' '!merged-packages/**' --ignore-path ../../.gitignore --log-level error",
     "publish:preview": "yarn npm publish --tag preview",
     "test": "vitest run --config vitest.config.ts",
     "test:e2e": "vitest run --config vitest.config.e2e.ts",
     "test:clean": "yarn test --no-cache --coverage.clean",
-    "test:dev": "yarn test --mode development",
+    "test:dev": "yarn test --mode development --reporter dot",
     "test:verbose": "yarn test --reporter verbose",
     "test:watch": "vitest --config vitest.config.ts"
   },