chore(core): point wren-core-py / wren / wren-core-wasm at Canner/WrenAI#2230
Conversation
After the import, the canonical repo URL is Canner/WrenAI rather than
the now-archived Canner/wren-engine. Updates the package manifests so
PyPI / npm / crates.io display the correct Repository / Issues links:
- core/wren-core-py/Cargo.toml repository
- core/wren-core-py/pyproject.toml Repository, Issues
- core/wren/pyproject.toml Repository, Issues
- core/wren-core-wasm/Cargo.toml repository
- core/wren-core-wasm/package.json repository.url + repository.directory
(now core/wren-core-wasm under the new layout)
CHANGELOG entries that link to historical wren-engine PR numbers/commits
are intentionally left alone — those resolve against the archived
wren-engine repo and would dangle if rewritten to WrenAI's renumbered
PR space.
WalkthroughRepository and package metadata URLs were updated across manifests and docs to point from ChangesRepository Metadata Migration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.Comment |
Adopt the @WrenAI npm org scope so the package lives under a namespace we own, and add publishConfig.access=public so first publish doesn't trip on scoped packages defaulting to private. Update README, AGENT_GUIDE and the CDN example to match the new import path. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@core/wren-core-wasm/examples/test-cdn.html`:
- Line 41: The import in test-cdn.html (import { WrenEngine } from
'https://unpkg.com/@wrenai/wren-core-wasm@0.1.0/dist/index.js') points to an
unpublished package and will 404; either publish the
`@wrenai/wren-core-wasm`@0.1.0 package to npm so the unpkg URL resolves, or update
this import (and the same reference in README.md and AGENT_GUIDE.md) to a
published version or a local/dev CDN URL; ensure the chosen version string in
the import matches the package version you publish.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: CHILL
Plan: Pro
Run ID: cf150011-ed32-4210-8307-c1f14aea0907
📒 Files selected for processing (4)
core/wren-core-wasm/AGENT_GUIDE.mdcore/wren-core-wasm/README.mdcore/wren-core-wasm/examples/test-cdn.htmlcore/wren-core-wasm/package.json
✅ Files skipped from review due to trivial changes (2)
- core/wren-core-wasm/AGENT_GUIDE.md
- core/wren-core-wasm/README.md
🚧 Files skipped from review as they are similar to previous changes (1)
- core/wren-core-wasm/package.json
| // unpkg used instead of jsDelivr: jsDelivr's free CDN has a 50 MB per-file | ||
| // limit and the WASM binary is ~68 MB raw. unpkg handles larger files. | ||
| import { WrenEngine } from 'https://unpkg.com/wren-core-wasm@0.1.0/dist/index.js'; | ||
| import { WrenEngine } from 'https://unpkg.com/@wrenai/wren-core-wasm@0.1.0/dist/index.js'; |
There was a problem hiding this comment.
🧩 Analysis chain
🌐 Web query:
Is @wrenai/wren-core-wasm published on npm?
💡 Result:
No, @wrenai/wren-core-wasm is not published on npm.
Citations:
- 1: https://github.com/Canner/wren-engine/releases/tag/wren-v0.3.0
- 2: https://libraries.io/pypi/wren-core-py
- 3: https://pypi.org/project/wren-engine/
Publish @wrenai/wren-core-wasm@0.1.0 to npm before merging.
The unpkg CDN URL will fail to load (404) because @wrenai/wren-core-wasm@0.1.0 is not published on npm. This breaks the example code entirely. The same URL is referenced in README.md and AGENT_GUIDE.md, so publishing the package is a blocking dependency for this and other consumer-facing docs.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@core/wren-core-wasm/examples/test-cdn.html` at line 41, The import in
test-cdn.html (import { WrenEngine } from
'https://unpkg.com/@wrenai/wren-core-wasm@0.1.0/dist/index.js') points to an
unpublished package and will 404; either publish the
`@wrenai/wren-core-wasm`@0.1.0 package to npm so the unpkg URL resolves, or update
this import (and the same reference in README.md and AGENT_GUIDE.md) to a
published version or a local/dev CDN URL; ensure the chosen version string in
the import matches the package version you publish.
Summary
After the wren-engine import, the canonical repo URL is
Canner/WrenAI. The published package manifests still pointed atCanner/wren-engine. Updating them so PyPI / npm / crates.io show the correct Repository and Issues links.Changes
core/wren-core-py/Cargo.tomlrepositorycore/wren-core-py/pyproject.tomlRepository,Issuescore/wren/pyproject.tomlRepository,Issuescore/wren-core-wasm/Cargo.tomlrepositorycore/wren-core-wasm/package.jsonrepository.url+repository.directory(nowcore/wren-core-wasm)Intentionally NOT changed
CHANGELOG.mdhistorical links (core/wren-core-py/CHANGELOG.md,core/wren/CHANGELOG.md) — those reference real wren-engine PR numbers and commits. The archivedCanner/wren-enginerepo will keep resolving them; rewriting toCanner/WrenAIwould dangle (different PR numbering on the new repo).core/wren-core/Cargo.toml— workspace root, norepositoryfield set; sub-crates inherit nothing for this field.🤖 Generated with Claude Code
Summary by CodeRabbit