Skip to content

revert: drop npm distribution again (third 403, definitively closed) - #106

Merged
yanacuti1121 merged 1 commit into
mainfrom
revert/npm-scoped-package
Aug 1, 2026
Merged

revert: drop npm distribution again (third 403, definitively closed)#106
yanacuti1121 merged 1 commit into
mainfrom
revert/npm-scoped-package

Conversation

@yanacuti1121

@yanacuti1121 yanacuti1121 commented Aug 1, 2026

Copy link
Copy Markdown
Owner

Summary

Reverts PR #105. The re-attempt (scoped @yanacuti/yana-ai, same original account) reached a real registry PUT in CI this time — got the same 403 as every prior attempt. The earlier optimism (a local dry-run passing the EPRIVATE client-side check) turned out to be a false signal, not evidence the account was healthy; see VERSIONING.md's new 2026-08-01 entry for the full writeup and why this should not be re-attempted without new evidence from npm support.

  • package.json: back to "private": true, name back to yana-ai
  • VERSIONING.md: reverted + new entry documenting this third attempt
  • All 4 READMEs: reverted to the not-distributed-via-npm state
  • .github/workflows/publish.yml: publish-npm job removed, header comment updated

Generated with Claude Code

Summary by CodeRabbit

  • Thay đổi phân phối

    • Ngừng phát hành sản phẩm qua npm.
    • Việc phát hành hiện chỉ còn được thực hiện qua PyPI và crates.io.
    • Gói npm được chuyển sang trạng thái riêng tư, phục vụ sử dụng nội bộ.
  • Tài liệu

    • Cập nhật hướng dẫn cài đặt bằng pipcargo.
    • Làm rõ sản phẩm không còn được phân phối trên npm.
    • Cập nhật quy tắc phiên bản và tag theo các kênh phát hành hiện có.

@codesandbox

codesandbox Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@cloudflare-workers-and-pages

Copy link
Copy Markdown
Contributor

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
yana b6259a2 Commit Preview URL

Branch Preview URL
Aug 01 2026, 12:59 AM

@coderabbitai

coderabbitai Bot commented Aug 1, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: be272207-f245-4d00-9cc0-a93913c13243

📥 Commits

Reviewing files that changed from the base of the PR and between 5470322 and b6259a2.

📒 Files selected for processing (7)
  • .github/workflows/publish.yml
  • README.ko.md
  • README.md
  • README.vi.md
  • README.zh.md
  • VERSIONING.md
  • package.json

📝 Walkthrough

Walkthrough

PR loại bỏ npm khỏi workflow publish, metadata package, mô hình versioning và hướng dẫn cài đặt. PyPI và crates.io vẫn được giữ lại.

Changes

Loại bỏ phân phối npm

Layer / File(s) Summary
Cập nhật metadata và mô hình phiên bản
package.json, VERSIONING.md
Package được đánh dấu private. Tài liệu loại bỏ product registry npm và chỉ giữ prefix rt-v*, py-v*.
Loại bỏ luồng publish npm
.github/workflows/publish.yml
Workflow chỉ giữ PyPI và crates.io. Các tag, registry và input npm bị xóa.
Đồng bộ hướng dẫn cài đặt
README*.md
Các README loại bỏ lệnh cài đặt npm và ghi rõ Product không có registry npm.

Estimated code review effort: 2 (Simple) | ~10 phút

Possibly related PRs

Poem

Thỏ con thấy npm khép cửa,
PyPI và crates.io vẫn sáng.
Tag mới chạy theo đường rõ ràng,
README chỉ lối cài đặt an toàn.
Package private, cà rốt reo vang!

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch revert/npm-scoped-package

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@yanacuti1121
yanacuti1121 merged commit 26c787b into main Aug 1, 2026
4 of 6 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