Skip to content

fix: add root to pnpm workspace for changeset resolution#68

Merged
himerus merged 2 commits intomainfrom
staging
Mar 14, 2026
Merged

fix: add root to pnpm workspace for changeset resolution#68
himerus merged 2 commits intomainfrom
staging

Conversation

@himerus
Copy link
Copy Markdown
Contributor

@himerus himerus commented Mar 14, 2026

Summary

  • Adds . (root) to pnpm-workspace.yaml packages list so changesets can resolve the helixir package

Why

The Publish workflow failed on PR #67 merge because changesets couldn't find helixir as a workspace member. The workspace file only listed packages/* — the root package wasn't visible to changesets.

After merging

The changeset bot will successfully run changeset version, create a version PR bumping helixir to 0.2.0, and the publish pipeline will work.

Summary by CodeRabbit

  • Chores
    • Updated workspace configuration to include the root directory as a workspace in the monorepo setup.

himerus and others added 2 commits March 13, 2026 21:59
…ixir

pnpm-workspace.yaml was restored for npm audit overrides but excluded
the root package. Changesets requires workspace members to be listed —
adding '.' lets it find the helixir package for version bumps.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Mar 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 82ded7c5-4297-4ee5-a386-1336b6a6b25b

📥 Commits

Reviewing files that changed from the base of the PR and between 36f5b34 and 690aaf9.

📒 Files selected for processing (1)
  • pnpm-workspace.yaml

Walkthrough

Modifies pnpm-workspace.yaml configuration to include the repository root ('.') as a workspace while removing the 'packages/*' glob pattern, altering how pnpm discovers and treats workspace directories.

Changes

Cohort / File(s) Summary
pnpm Workspace Configuration
pnpm-workspace.yaml
Added root directory ('.') to pnpm workspaces and removed 'packages/*' glob pattern, changing workspace resolution from subdirectories to root-level inclusion.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely summarizes the main change: adding root to pnpm workspace to enable changeset resolution.
Description check ✅ Passed The description includes a clear summary of changes and rationale; however, it lacks the structured template fields like Type of change, Tests added, and Checklist items.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch staging
📝 Coding Plan
  • Generate coding plan for human review comments

Comment @coderabbitai help to get the list of available commands and usage tips.

@himerus himerus merged commit cc49f83 into main Mar 14, 2026
15 of 16 checks passed
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