Skip to content

chore: add iterm2, bun, and agent tooling tweaks#46

Merged
akornmeier merged 2 commits into
mainfrom
feat/agent-iterm-setup
Apr 27, 2026
Merged

chore: add iterm2, bun, and agent tooling tweaks#46
akornmeier merged 2 commits into
mainfrom
feat/agent-iterm-setup

Conversation

@akornmeier
Copy link
Copy Markdown
Owner

@akornmeier akornmeier commented Apr 27, 2026

Summary

  • Brewfile: add bun and iterm2; pin claude-code@latest
  • gitignore: add common dev ignores (node_modules/, dist/, .env*, *.log)
  • package.json: bump pnpm to 10.33.0
  • tmux aliases: add --verbose to cldyo
  • zshrc: fix zsh-completions fpath via brew --prefix

Test plan

  • dot update succeeds
  • brew bundle check passes after install
  • New shell loads completions without errors
  • cldyo alias launches with verbose output

🤖 Generated with Claude Code

akornmeier and others added 2 commits April 26, 2026 21:24
- Brewfile: add bun, iterm2; pin claude-code@latest
- gitignore: add common dev ignores (node_modules, dist, .env*, *.log)
- package.json: bump pnpm to 10.33.0
- path.zsh: add trailing newline
- tmux aliases: add --verbose to cldyo
- zshrc: fix zsh-completions fpath via brew --prefix

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Removes hardcoded version: 10.6.5 from pnpm/action-setup steps so the
action picks up the version from package.json's packageManager field
(currently 10.33.0). Resolves ERR_PNPM_BAD_PM_VERSION on CI.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@akornmeier akornmeier requested a review from Copilot April 27, 2026 01:28
@akornmeier akornmeier merged commit 11ff061 into main Apr 27, 2026
4 checks passed
@akornmeier akornmeier deleted the feat/agent-iterm-setup branch April 27, 2026 01:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the dotfiles/tooling setup to add new Homebrew packages, adjust shell completion initialization, expand global gitignore patterns, and align CI’s pnpm setup with the packageManager field.

Changes:

  • Add bun and iterm2 to the Brewfile and adjust the Claude Code cask entry.
  • Update global gitignore entries for common Node/dev artifacts.
  • Bump pnpm in package.json and remove hardcoded pnpm versions from GitHub Actions workflows.

Reviewed changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
zsh/zshrc.symlink Changes compinit/FPATH initialization to use brew --prefix and gates completion init on brew.
tmux/aliases.zsh Adds --verbose to the cldyo alias.
system/path.zsh No functional change (appears to be a formatting-only change).
package.json Bumps packageManager pnpm version to 10.33.0 (with integrity).
git/gitignore.symlink Adds node_modules/, dist/, .env*, *.log but removes several prior ignores.
Brewfile Adds bun and iterm2; changes claude-code to claude-code@latest.
.github/workflows/release.yml Removes hardcoded pnpm version in setup step.
.github/workflows/commitlint.yml Removes hardcoded pnpm version in setup step.
.github/workflows/changeset-check.yml Removes hardcoded pnpm version in setup step.
.changeset/iterm-agent-tooling.md Adds a changeset describing these tooling updates.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread zsh/zshrc.symlink
Comment on lines +79 to +80
# autoload -Uz compinit
# compinit -i
Copy link

Copilot AI Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The previous compinit initialization is still present as commented-out code. Since the new logic is directly below, consider deleting these commented lines to reduce noise and avoid ambiguity about the intended initialization sequence.

Copilot uses AI. Check for mistakes.
Comment thread git/gitignore.symlink
.env
.env.*
*.log
.DS_Store
Copy link

Copilot AI Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change removes previously ignored patterns (e.g. editor backup *~, swap *.swp, and .netlify). That will cause these files to start showing up in git status across all repos using this global ignore. If the intent is to “expand” ignores, consider re-adding the removed entries alongside the new ones.

Suggested change
.DS_Store
.DS_Store
*~
*.swp
.netlify

Copilot uses AI. Check for mistakes.
Comment thread zsh/zshrc.symlink
Comment on lines +82 to +86
if type brew &>/dev/null; then
FPATH=$(brew --prefix)/share/zsh-completions:$FPATH
autoload -Uz compinit
compinit
fi
Copy link

Copilot AI Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

compinit is now only executed when brew is available, which means the completion system won’t be initialized on machines/shells without Homebrew even though the rest of the config still loads */completion.zsh files. Also, calling compinit without -i diverges from existing usage in this repo (e.g. git/completion.zsh) and can re-introduce insecure-dir warnings that prevent completions from loading. Consider running autoload -Uz compinit && compinit -i unconditionally (or gated on autoload/compinit availability), and only prepend the Homebrew zsh-completions path when the directory exists.

Copilot uses AI. Check for mistakes.
Comment thread zsh/zshrc.symlink
Comment on lines +74 to +76
# if [[ -d "$HOMEBREW_PREFIX/share/zsh-completions" ]]; then
# FPATH="$HOMEBREW_PREFIX/share/zsh-completions:$FPATH"
# fi
Copy link

Copilot AI Apr 27, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These commented-out lines look like dead code from the previous FPATH approach. Keeping the old implementation commented in the main rc file makes it harder to maintain and can be confusing later; consider removing it entirely (or converting it into a short explanatory comment about why brew --prefix is used instead).

Copilot uses AI. Check for mistakes.
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.

2 participants