Skip to content

fix: include README and LICENSE in the published package - #267

Merged
uhyo merged 1 commit into
masterfrom
claude/friendly-planck-a30ext
Oct 8, 2026
Merged

uhyo merged 1 commit into
masterfrom
claude/friendly-planck-a30ext

Conversation

@uhyo

@uhyo uhyo commented Oct 8, 2026

Copy link
Copy Markdown
Owner

Summary

The @funstack/router npm package didn't include a README (or LICENSE). Both files live at the repository root, but npm only picks them up from the package directory (packages/router).

  • Add a prepack script to packages/router that copies the root README.md and LICENSE into the package directory. pnpm publish runs prepack automatically, so publish.yml doesn't need any changes.
  • Gitignore the copied files in packages/router/.gitignore.
  • Change the hero image in the root README to an absolute raw.githubusercontent.com URL so it renders on npmjs.com. The relative path wouldn't load there because the image isn't in the package.

Test plan

  • pnpm pack in packages/router produces a tarball containing README.md and LICENSE
  • pnpm format:check passes

🤖 Generated with Claude Code

https://claude.ai/code/session_01XzEsqHjb7nDGfaePZYkyYk


Generated by Claude Code

Copy the repository root README.md and LICENSE into packages/router on
prepack so they are included in the npm tarball. The hero image in the
README now uses an absolute URL so it renders on npmjs.com.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01XzEsqHjb7nDGfaePZYkyYk
@uhyo
uhyo merged commit 40b3e8c into master Oct 8, 2026
1 check passed
@uhyo
uhyo deleted the claude/friendly-planck-a30ext branch October 8, 2026 13:44
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