Skip to content

Add npm CLI entrypoint#1149

Merged
frankslin merged 2 commits into
BYVoid:masterfrom
frankslin:upstream-master
Apr 26, 2026
Merged

Add npm CLI entrypoint#1149
frankslin merged 2 commits into
BYVoid:masterfrom
frankslin:upstream-master

Conversation

@frankslin
Copy link
Copy Markdown
Collaborator

Expose a package bin named opencc that provides basic text conversion through the existing Node.js binding without changing the native addon API.

Support stdin/stdout, input and output files, config selection, help, and version output. Explicitly reject unsupported diagnostic/plugin features such as --inspect, --segmentation, --path, and measured results.

Document the global npm CLI usage and add smoke tests for conversion, help, version, and unsupported modes.

Expose a package bin named opencc that provides basic text conversion through the existing Node.js binding without changing the native addon API.

Support stdin/stdout, input and output files, config selection, help, and version output. Explicitly reject unsupported diagnostic/plugin features such as --inspect, --segmentation, --path, and measured results.

Document the global npm CLI usage and add smoke tests for conversion, help, version, and unsupported modes.
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 53aeebe889

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread node/cli.js Outdated
Comment thread node/cli.js Outdated
@frankslin frankslin merged commit 8a1ec6c into BYVoid:master Apr 26, 2026
32 checks passed
@frankslin frankslin deleted the upstream-master branch April 26, 2026 22:31
@frankslin frankslin mentioned this pull request May 9, 2026
31 tasks
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