Skip to content

chore: remove version pinning for napkin-ai - #9

Open
cad0p wants to merge 1 commit into
Michaelliv:mainfrom
cad0p:chore/remove-napkin-ai-version-pin
Open

cad0p wants to merge 1 commit into
Michaelliv:mainfrom
cad0p:chore/remove-napkin-ai-version-pin

Conversation

@cad0p

@cad0p cad0p commented May 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Change napkin-ai dependency from ^0.8.1 to * (any version).

Rationale

napkin-ai is a sibling package maintained in the same organization, not an external dependency. Version pinning adds unnecessary friction when napkin-ai ships new features that pi-napkin depends on — requiring a coordinated version bump in both packages.

With *, pi-napkin always uses the latest napkin-ai, which is the intended behavior for tightly coupled packages.

Companion PRs

  • napkin#15 — adds global vault config fallback
  • napkin#16 — adds prepare script for git dependency builds
  • pi-napkin#8 — uses napkin's native global config

napkin-ai is a sibling package maintained in the same organization,
not an external dependency. Removing the version pin avoids
unnecessary friction when napkin-ai gets new features that pi-napkin
relies on.
@cad0p
cad0p marked this pull request as ready for review May 4, 2026 00:18
Copilot AI review requested due to automatic review settings May 4, 2026 00:18

Copilot AI left a comment

Copy link
Copy Markdown

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 removes version pinning for the napkin-ai runtime dependency so pi-napkin can pick up newer napkin-ai releases without coordinated version bumps.

Changes:

  • Update napkin-ai dependency specifier from ^0.8.1 to * in package.json.

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

Comment thread package.json
},
"dependencies": {
"napkin-ai": "^0.8.1"
"napkin-ai": "*"
Comment thread package.json
},
"dependencies": {
"napkin-ai": "^0.8.1"
"napkin-ai": "*"
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