Skip to content

chore: rename package to @pokujs/react#11

Merged
Lojhan merged 5 commits intomainfrom
copilot/adjust-package-name-to-pokujs
Apr 3, 2026
Merged

chore: rename package to @pokujs/react#11
Lojhan merged 5 commits intomainfrom
copilot/adjust-package-name-to-pokujs

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 2, 2026

  • Rename package metadata (name, repository, homepage, bugs) from poku-react-testing to @pokujs/react
  • Update Symbol.for key and log/error message prefixes in source files
  • Update README.md, benchmark/REPORT.md references
  • Update benchmark/package.json and benchmark/poku.config.js
  • Update benchmark/run.mjs (comments, syncLocalBuild path, template strings)
  • Update all benchmark/tests/poku/*.jsx imports
  • Update cd_publish.yml package-name and codeql-config.yml name
  • Regenerate lock files
  • Remove redundant NODE_AUTH_TOKEN check from publish step (npm handles this natively)
  • Restore CHANGELOG.md to preserve original Lojhan/poku-react-testing history links
Original prompt

I'm moving this repo to poku organization (@pokujs) under the name of @pokujs/react. Create a PR adjusting the package name etc to match it.

Copilot AI changed the title [WIP] Update package name to match new organization chore: rename package to @pokujs/react Apr 2, 2026
Copilot AI requested a review from Lojhan April 2, 2026 14:12
@Lojhan
Copy link
Copy Markdown
Collaborator

Lojhan commented Apr 2, 2026

@copilot can you change the cd npm publish step to expect the auth token?

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 2, 2026

@copilot can you change the cd npm publish step to expect the auth token?

Done in f4948dc. The publish step now checks for NODE_AUTH_TOKEN before running npm publish, and exits with a clear error message if the NPM_TOKEN secret isn't configured.

Copilot AI requested a review from Lojhan April 2, 2026 14:52
@Lojhan Lojhan marked this pull request as ready for review April 2, 2026 14:52
@Lojhan
Copy link
Copy Markdown
Collaborator

Lojhan commented Apr 3, 2026

@copilot can you fix the comments from @wellwelwel?

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 3, 2026

@copilot can you fix the comments from @wellwelwel?

Fixed in f31e331:

  • Removed the redundant NODE_AUTH_TOKEN check from the publish step — npm already surfaces a clear auth error on its own.
  • Restored CHANGELOG.md to its original Lojhan/poku-react-testing links to preserve the project history.

Copilot AI requested a review from Lojhan April 3, 2026 14:24
@Lojhan
Copy link
Copy Markdown
Collaborator

Lojhan commented Apr 3, 2026

@copilot what about formatting the md and other stuff in the /benchmark dir?

Copy link
Copy Markdown
Contributor Author

Copilot AI commented Apr 3, 2026

@copilot what about formatting the md and other stuff in the /benchmark dir?

Done in 6260ad6. Updated in the benchmark directory:

  • All 5 benchmark/tests/poku/*.jsx imports changed from poku-react-testing to @pokujs/react
  • benchmark/run.mjs: updated comments, syncLocalBuild destination path (node_modules/@pokujs/react), log message, and the template strings for REPORT.md generation
  • benchmark/REPORT.md: fixed the "Frameworks under test" table column alignment (the rows now all line up to the same 35-char column width)

@Lojhan Lojhan merged commit 59b6baf into main Apr 3, 2026
10 checks passed
@Lojhan Lojhan deleted the copilot/adjust-package-name-to-pokujs branch April 3, 2026 14:51
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.

3 participants