Skip to content

rebrand: remove remaining @inferedge/moss and inferedge_moss references#161

Open
ashvathsureshkumar wants to merge 7 commits intomainfrom
rebrand/inferedge-to-moss
Open

rebrand: remove remaining @inferedge/moss and inferedge_moss references#161
ashvathsureshkumar wants to merge 7 commits intomainfrom
rebrand/inferedge-to-moss

Conversation

@ashvathsureshkumar
Copy link
Copy Markdown
Contributor

@ashvathsureshkumar ashvathsureshkumar commented Apr 20, 2026

Summary

Full sweep of leftover old-name references left in this repo after the rename from Inferedge to Moss. 62 files changed across READMEs, demo-app manifests, Python and TS/JS imports, Vue assets, generated TypeDoc reference docs in the vitepress-plugin-moss demo-site, and dead InferEdge logo assets.

Renames applied

From To
@inferedge/moss (npm) @moss-dev/moss
inferedge_moss (Python import) moss
inferedge-moss (PyPI dep) moss
@anthropic-ai/moss-core (copy-paste typo in sdks/javascript/bindings/Cargo.toml) @moss-dev/moss-core
pepy.tech badge inferedge-moss-core (root + org-profile README) moss
npm badge @inferedge/moss (root + org-profile README) @moss-dev/moss
https://docs.usemoss.dev (prose) https://docs.moss.dev
https://usemoss.dev / https://www.usemoss.dev (root-domain prose only) https://moss.dev
github.com/usemoss/moss-samples (404s) github.com/usemoss/moss
InferEdgeLogo_Dark_Icon.png (packages/vitepress-plugin-moss/ + moss-live-labs/examples/image-search/react-app/) canonical assets/moss-logo.png / existing Moss-logo.jpeg; dead PNGs removed
inferedge_moss/__init__.pyi path in MAINTAINERS.md actual path sdks/python/sdk/src/moss/__init__.pyi

Generated TypeDoc reference docs under packages/vitepress-plugin-moss/demo-site/documentation/docs/reference/{js,python}/ had their header text, breadcrumb links, and install-command snippets bulk-renamed. The next regeneration run will overwrite these anyway; this just stops them from shipping stale.

Not changed (intentional, flag for follow-up)

  • inferedge-moss-core Rust-bindings distribution name in sdks/python/bindings/pyproject.toml.
  • inferedge-moss-core==0.8.7 transitive dep pin in sdks/python/sdk/pyproject.toml. moss-core on PyPI is only at 0.2.3 and the API is incompatible.
  • @inferedge-rest/moss in packages/moss-md-indexer/src/uploader.ts and in README dependency-split prose — target @moss-dev/moss-rest doesn't exist yet on npm.
  • InferEdge Inc. legal-entity references in copyright comments and in apps/moss-llamaindex/frontend/components/common/footer.tsx.
  • portal.usemoss.dev and service.usemoss.dev subdomains — leave per rebrand decision.
  • Historical "previously published as inferedge-moss" / "Initial release of @inferedge/moss" lines in the two SDK changelogs — intentional past-tense migration context.

Verification

  • apps/next-js: fresh npm install installs @moss-dev/moss cleanly; npx tsc --noEmit passes.
  • Full-repo grep for @inferedge/moss | inferedge_moss | inferedge-moss (excluding -core, the two SDK changelogs, @inferedge-rest/moss, and node_modules / lockfiles) returns zero hits.
  • docs.usemoss.dev returns zero prose hits.
  • InferEdgeLogo returns zero references from code (the remaining match is an incidental byte sequence inside apps/moss-llamaindex/frontend/public/Icon.png binary data).

Audit context

Part of the full-surface Moss rebrand audit. Sibling PRs:


Open in Devin Review

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 3 potential issues.

View 1 additional finding in Devin Review.

Open in Devin Review

Comment thread packages/vitepress-plugin-moss/package.json
Comment thread packages/vitepress-plugin-moss/package.json Outdated
## 1. Go to Moss Portal and Get API Keys

- Visit [usemoss.dev](https://usemoss.dev/) to sign up, create an account, confirm your email, and sign in.
- Visit [moss.dev](https://moss.dev/) to sign up, create an account, confirm your email, and sign in.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚩 Partial URL migration: portal.usemoss.dev references remain across many files

The PR updates some usemoss.dev URLs to moss.dev (e.g., in getting-started.md:11, image-search/readme.md:43), but many files outside the diff still reference portal.usemoss.dev — including apps/pipecat-moss/pipecat-quickstart/README.md:20, apps/next-js/README.md:23, apps/livekit-moss-vercel/.../README.md:12, packages/pipecat-moss/README.md:14, Python bot scripts, and the LlamaIndex frontend. If the portal URL has genuinely changed, these are incomplete migrations. If portal.usemoss.dev is still valid (perhaps separate from the main site), this is intentional. Worth confirming whether the portal domain has migrated too.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR completes the repo-wide rebrand cleanup by replacing remaining legacy Inferedge package/import/doc references with the new Moss names and URLs, including SDK dependency updates and logo asset swaps.

Changes:

  • Updated JS/TS and Python-facing documentation snippets and imports to use @moss-dev/moss / moss.
  • Replaced old portal/docs/samples URLs with moss.dev / docs.moss.dev.
  • Swapped remaining InferEdge logo assets for Moss equivalents and updated references.

Reviewed changes

Copilot reviewed 57 out of 62 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
sdks/javascript/bindings/Cargo.toml Fixes npm install comment to the correct @moss-dev/moss-core package.
packages/vitepress-plugin-moss/package.json Updates published files list and switches dependency to @moss-dev/moss.
packages/vitepress-plugin-moss/moss-logo.png Adds Moss logo asset used by the plugin UI.
packages/vitepress-plugin-moss/demo-site/documentation/docs/reference/** Bulk-renames generated reference docs headers/breadcrumbs/install snippets to the new package names.
packages/vitepress-plugin-moss/demo-site/documentation/docs/{index,getting-started}.md Updates portal URLs and install/import examples to new Moss package names.
packages/vitepress-plugin-moss/demo-site/documentation/MAINTAINERS.md Updates the listed Python reference-docs source path (but command/path details need correction—see comment).
packages/vitepress-plugin-moss/Search.vue Updates type/runtime imports to @moss-dev/moss and swaps the logo import.
packages/vitepress-plugin-moss/README.md Updates runtime SDK reference to @moss-dev/moss.
packages/pipecat-moss/README.md Updates docs URL to docs.moss.dev.
moss-live-labs/examples/image-search/readme.md Updates portal URL to moss.dev.
moss-live-labs/examples/image-search/react-app/src/pages/ItemsPage.tsx Updates logo paths to Moss asset.
apps/pipecat-moss/pipecat-quickstart/README.md Updates docs URL to docs.moss.dev.
apps/next-js/package.json Switches dependency from @inferedge/moss to @moss-dev/moss.
apps/next-js/package-lock.json Updates lockfile to reflect the new SDK package and dependency graph changes.
apps/next-js/app/actions.ts Updates import to @moss-dev/moss.
apps/next-js/app/actions.test.ts Updates module mock to @moss-dev/moss.
apps/next-js/README.md Updates install instructions to @moss-dev/moss.
apps/livekit-moss-vercel/livekit-voice-agent/livekit-moss-agent/README.md Updates Moss link to moss.dev.
README.md Updates badges/links to new npm/PyPI identifiers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

| --- | --- | --- |
| `docs/reference/js/**/*` | `javascript/user-facing-sdk/typedoc.json` | `npm run docs` (inside `javascript/user-facing-sdk/`) |
| `docs/reference/python/**/*` | `python/user-facing-sdk/src/inferedge_moss/__init__.pyi` | `python scripts/generate_docs.py` (inside `python/user-facing-sdk/`) |
| `docs/reference/python/**/*` | `sdks/python/sdk/src/moss/__init__.pyi` | `python scripts/generate_docs.py` (inside `sdks/python/sdk/`) |
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

The MAINTAINERS table now points Python reference docs generation at sdks/python/sdk/src/moss/__init__.pyi, but the command python scripts/generate_docs.py (inside sdks/python/sdk/) appears to be incorrect: there is no scripts/ directory or generate_docs.py in sdks/python/sdk/. Please update this entry (and the regeneration workflow section below, which still references python/user-facing-sdk) to the actual path/command used to regenerate docs/reference/python/**/* from the current Python SDK.

Copilot uses AI. Check for mistakes.
Comment thread README.md Outdated
[![License](https://img.shields.io/badge/License-BSD_2--Clause-orange.svg)](https://opensource.org/licenses/BSD-2-Clause)
[![PyPI](https://img.shields.io/pypi/v/moss?color=deepgreen)](https://pypi.org/project/moss/)
[![PyPI downloads](https://static.pepy.tech/personalized-badge/inferedge-moss-core?period=total&units=international_system&left_color=grey&right_color=blue&left_text=pypi+downloads)](https://pepy.tech/project/inferedge-moss-core)
[![PyPI downloads](https://static.pepy.tech/personalized-badge/moss?period=total&units=international_system&left_color=grey&right_color=blue&left_text=pypi+downloads)](https://pepy.tech/project/moss)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We will be using inferedge-moss-core as the core package downloads indicator.

Comment thread README.md Outdated
[![PyPI downloads](https://static.pepy.tech/personalized-badge/moss?period=total&units=international_system&left_color=grey&right_color=blue&left_text=pypi+downloads)](https://pepy.tech/project/moss)
[![npm](https://img.shields.io/npm/v/@moss-dev/moss?color=deepgreen)](https://www.npmjs.com/package/@moss-dev/moss)
[![npm downloads](https://img.shields.io/npm/dt/@inferedge/moss?label=npm+downloads&color=blue)](https://www.npmjs.com/package/@inferedge/moss)
[![npm downloads](https://img.shields.io/npm/dt/@moss-dev/moss?label=npm+downloads&color=blue)](https://www.npmjs.com/package/@moss-dev/moss)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Lets make this change once our @moss-dev/moss installs reach critical number and/or crosses our old package installs.

@vercel
Copy link
Copy Markdown
Contributor

vercel Bot commented Apr 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
moss_nextjs Ready Ready Preview, Comment Apr 21, 2026 1:47am

Request Review

devin-ai-integration[bot]

This comment was marked as resolved.

@yatharthk2
Copy link
Copy Markdown
Collaborator

next-js and vitepress are using the webpackage of moss.

@@ -1,8 +1,8 @@
[**@inferedge/moss v1.0.0-beta.7**](../README.md)
[**@moss-dev/moss v1.0.0-beta.7**](../README.md)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

beta version does not exist for this package

---

[inferedge-moss v1.0.0b16](../README.md)
[moss v1.0.0b16](../README.md)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

no beta version for this package

},
"dependencies": {
"@inferedge/moss": "^1.0.0-beta.7",
"@moss-dev/moss": "^1.0.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@moss-dev/moss-web needs to be used.
Please test the app before merging.

Comment thread apps/next-js/package.json Outdated
},
"dependencies": {
"@inferedge/moss": "^1.0.0-beta.7",
"@moss-dev/moss": "^1.0.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think we need @moss-dev/moss-web here

ashvathsureshkumar added a commit that referenced this pull request Apr 21, 2026
Three follow-ups from PR review:

1. Revert README badge slugs for PyPI downloads and npm downloads per
   @r4ghu. The download counters should continue to reflect the old
   `inferedge-moss-core` / `@inferedge/moss` packages until the new
   `moss` / `@moss-dev/moss` installs reach critical volume. Keeps the
   PyPI version + npm version badges (the top-of-page links) pointing
   at the new packages.

2. Revert the MAINTAINERS.md Python doc-source path change. The old
   entry pointed at `python/user-facing-sdk/src/inferedge_moss/__init__.pyi`
   and my previous edit moved it to `sdks/python/sdk/src/moss/__init__.pyi`,
   but Copilot correctly flagged that the regen command
   (`python scripts/generate_docs.py`) doesn't exist in the current
   repo — neither script nor script dir are present anywhere under
   `sdks/`. The whole regen workflow is stale in ways beyond this
   rebrand; leaving the path as it was so it's obviously-stale for the
   docs team to rewrite properly, rather than half-updated.

3. Regenerate `packages/vitepress-plugin-moss/pnpm-lock.yaml` via
   `pnpm install --lockfile-only`. The previous commit renamed the
   `@inferedge/moss` dep in package.json but didn't refresh the lock,
   so `pnpm install --frozen-lockfile` would have failed in CI (Devin
   flag). Lock now cleanly references `@moss-dev/moss` with zero
   `@inferedge` entries.

Not addressed (out of scope for this PR):
- `packages/vitepress-plugin-moss/demo-site/package-lock.json` has
  2 pre-existing `@inferedge/moss` refs. That file exists on `main`
  before this PR, the demo-site is a pnpm workspace (pnpm-lock.yaml
  is the active lockfile and is clean), and npm can't even regenerate
  it because it uses `link:..` for the workspace dep. Orphan from a
  prior npm-era install; leaving for separate cleanup.
- `portal.usemoss.dev` references throughout — intentionally
  preserved per the rebrand policy decision.
@ashvathsureshkumar
Copy link
Copy Markdown
Contributor Author

Addressed review feedback in ddd84ef:

@r4ghu — reverted both download-counter badge slugs per your guidance. README.md:13 (PyPI downloads) keeps inferedge-moss-core and README.md:15 (npm downloads) keeps @inferedge/moss. The top-of-page version badges (:12 PyPI v1.0.0 and :14 @moss-dev/moss v1.0.1) still point at the new packages so visitors see the current release. I'll open a follow-up to flip the counters once new-package installs cross the old-package totals.

@Copilot — good catch on MAINTAINERS.md. Verified: sdks/python/sdk/ has no scripts/ dir and no generate_docs.py anywhere in the repo, and the old python/user-facing-sdk/ path referenced in the regen-workflow section below also no longer exists. The whole Python doc-regen entry is stale in ways beyond this rebrand — reverting my path edit so it's obviously-stale rather than half-updated, and leaving a proper rewrite for the docs-tooling team.

@devin-ai-integration — regenerated packages/vitepress-plugin-moss/pnpm-lock.yaml via pnpm install --lockfile-only. Lock now has 18 @moss-dev/moss entries and zero @inferedge entries, so pnpm install --frozen-lockfile will pass.

The secondary packages/vitepress-plugin-moss/demo-site/package-lock.json flag: that file exists on main before this PR, the demo-site is a pnpm workspace with pnpm-lock.yaml as the active lockfile (already clean), and npm can't even regenerate it because it uses link:.. for the workspace dep. Orphan from a prior npm-era install — leaving for separate cleanup rather than expanding this PR's scope.

Partial-portal-URL-migration flag acknowledged but intentional: per the rebrand decision, portal.usemoss.dev and service.usemoss.dev stay put in this pass.

devin-ai-integration[bot]

This comment was marked as resolved.

@ashvathsureshkumar
Copy link
Copy Markdown
Contributor Author

Round 2 of review fixes in a85c35e.

@HarshaNalluru / @yatharthk2 — you were right, @moss-dev/moss-web is the correct package for the browser subtrees. Switched:

  • apps/next-js: package.json, app/actions.ts, app/actions.test.ts, README.md — now on @moss-dev/moss-web. Package-lock.json regenerated. Also fixed a pre-existing latent type mismatch that the new SDK surfaced: results.timeTakenInMstimeTakenMs (matches the new SearchResult shape).
  • packages/vitepress-plugin-moss: package.json, Search.vue (both the static type import and the dynamic import()), README.md dependency-split prose — now on @moss-dev/moss-web. pnpm-lock.yaml regenerated (3 @moss-dev/moss-web entries, 0 non-hyphen @moss-dev/moss).
  • packages/vitepress-plugin-moss/demo-site/documentation/docs/{getting-started,index}.md — JS install commands and imports switched to @moss-dev/moss-web.

@HarshaNalluru on the reference-doc version labels — fixed:

  • JS refs (reference/js/*.md): [**@moss-dev/moss v1.0.0-beta.7**][**@moss-dev/moss-web v1.0.0**] (moss-web has no betas on npm, only 1.0.0).
  • Python refs (reference/python/*.md): [**moss v1.0.0b16**][**moss v1.0.0**] (PyPI only has moss@1.0.0, no betas).

Verification:

  • cd apps/next-js && npm install && npx tsc --noEmit → clean against @moss-dev/moss-web@1.0.0
  • cd packages/vitepress-plugin-moss && pnpm install --lockfile-only → clean

All other @moss-dev/moss references across the repo (SDK source, moss-bun, docker/javascript, vercel-sdk, test fixtures, moss-live-labs TypeScript examples, root-README code snippet, examples/javascript/*) are Node contexts and intentionally stay on @moss-dev/moss. The rename only applies to the two browser subtrees you called out.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 3 new potential issues.

View 4 additional findings in Devin Review.

Open in Devin Review

Comment thread apps/next-js/app/actions.ts
napi-derive = "3.0.0"
# The `moss` crate is not included in this repository.
# Pre-built binaries are available on npm: npm install @anthropic-ai/moss-core
# Pre-built binaries are available on npm: npm install @moss-dev/moss-core
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🚩 Cargo.toml comment references @moss-dev/moss-core — may need verification

The comment in sdks/javascript/bindings/Cargo.toml:18 was changed from @anthropic-ai/moss-core to @moss-dev/moss-core. This is a comment-only change and doesn't affect the build, but @moss-dev/moss-core should be verified as the correct published npm package name for the pre-built native binaries. The rest of the PR uses @moss-dev/moss-web for the high-level SDK, so @moss-dev/moss-core for the low-level bindings seems plausible but worth confirming.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Sweep of leftover old-name references after the rename to Moss. Touches
READMEs, badges, demo-app manifests, import statements across TS/JS/Vue
and Python, generated TypeDoc reference docs in the vitepress plugin
demo-site, and dead InferEdge logo assets.

- @inferedge/moss -> @moss-dev/moss
- inferedge_moss -> moss (Python imports)
- inferedge-moss -> moss (PyPI deps)
- @anthropic-ai/moss-core copy-paste typo -> @moss-dev/moss-core
  (sdks/javascript/bindings/Cargo.toml)
- pepy.tech badge slug inferedge-moss-core -> moss (root README + org
  profile README badges)
- npmjs.com/package/@inferedge/moss -> @moss-dev/moss (badge links)
- docs.usemoss.dev -> docs.moss.dev (prose-only)
- usemoss.dev root-domain prose links -> moss.dev
- github.com/usemoss/moss-samples (404s) -> github.com/usemoss/moss
- InferEdgeLogo_Dark_Icon.png (vitepress plugin + moss-live-labs image
  search) replaced with the canonical assets/moss-logo.png; dead PNGs
  removed
- MAINTAINERS.md doc-source path updated to match current layout
  (sdks/python/sdk/src/moss/)
- Generated TypeDoc reference docs under packages/vitepress-plugin-moss/
  demo-site/documentation/docs/reference/{js,python}/ — bulk-renamed
  header and breadcrumb text

- inferedge-moss-core Rust bindings distribution name in
  sdks/python/bindings/pyproject.toml
- inferedge-moss-core==0.8.7 transitive dep pin in sdks/python/sdk/
  pyproject.toml (moss-core on PyPI is only at 0.2.3 and has an
  incompatible API)
- @inferedge-rest/moss (target @moss-dev/moss-rest package doesn't
  exist yet; left in README dependency-split prose and in
  packages/moss-md-indexer/src/uploader.ts)
- InferEdge Inc. legal-entity references in copyright comments and in
  apps/moss-llamaindex/frontend/components/common/footer.tsx
- portal.usemoss.dev and service.usemoss.dev subdomains
- Historical changelog lines ('previously published as inferedge-moss',
  'Initial release of @inferedge/moss') in sdks/python/sdk/CHANGELOG.md
  and sdks/javascript/sdk/CHANGELOG.md

- apps/next-js: clean npm install + tsc --noEmit passes against the
  new @moss-dev/moss dep
- Full-repo grep for remaining stale strings (excluding -core, the
  historical changelogs, @inferedge-rest, and node_modules/lockfiles)
  returns zero hits
The previous sweep rewrote `github.com/usemoss/moss-samples/blob/main/
{python,javascript}/*` to `github.com/usemoss/moss/blob/main/
{python,javascript}/*`. Those paths 404 because the sample files live
under `examples/` in the current repo layout. Corrected to
`examples/{python,javascript}/*` — all four paths now return 200.
Three follow-ups from PR review:

1. Revert README badge slugs for PyPI downloads and npm downloads per
   @r4ghu. The download counters should continue to reflect the old
   `inferedge-moss-core` / `@inferedge/moss` packages until the new
   `moss` / `@moss-dev/moss` installs reach critical volume. Keeps the
   PyPI version + npm version badges (the top-of-page links) pointing
   at the new packages.

2. Revert the MAINTAINERS.md Python doc-source path change. The old
   entry pointed at `python/user-facing-sdk/src/inferedge_moss/__init__.pyi`
   and my previous edit moved it to `sdks/python/sdk/src/moss/__init__.pyi`,
   but Copilot correctly flagged that the regen command
   (`python scripts/generate_docs.py`) doesn't exist in the current
   repo — neither script nor script dir are present anywhere under
   `sdks/`. The whole regen workflow is stale in ways beyond this
   rebrand; leaving the path as it was so it's obviously-stale for the
   docs team to rewrite properly, rather than half-updated.

3. Regenerate `packages/vitepress-plugin-moss/pnpm-lock.yaml` via
   `pnpm install --lockfile-only`. The previous commit renamed the
   `@inferedge/moss` dep in package.json but didn't refresh the lock,
   so `pnpm install --frozen-lockfile` would have failed in CI (Devin
   flag). Lock now cleanly references `@moss-dev/moss` with zero
   `@inferedge` entries.

Not addressed (out of scope for this PR):
- `packages/vitepress-plugin-moss/demo-site/package-lock.json` has
  2 pre-existing `@inferedge/moss` refs. That file exists on `main`
  before this PR, the demo-site is a pnpm workspace (pnpm-lock.yaml
  is the active lockfile and is clean), and npm can't even regenerate
  it because it uses `link:..` for the workspace dep. Orphan from a
  prior npm-era install; leaving for separate cleanup.
- `portal.usemoss.dev` references throughout — intentionally
  preserved per the rebrand policy decision.
@HarshaNalluru and @yatharthk2 pointed out that the correct npm package
for browser / WebAssembly usage is `@moss-dev/moss-web` (not the Node
SDK `@moss-dev/moss`). Applied the correction in the two subtrees that
run in a browser:

- apps/next-js: package.json dep, actions.ts import, actions.test.ts
  vi.mock, README install command -> @moss-dev/moss-web. Regenerated
  package-lock.json. Also fixed a pre-existing type mismatch now
  surfaced by the new SDK: `results.timeTakenInMs` -> `timeTakenMs`
  (the new SearchResult shape).
- packages/vitepress-plugin-moss: package.json dep, Search.vue (static
  + dynamic imports), README dependency-split prose ->
  @moss-dev/moss-web. Regenerated pnpm-lock.yaml.
- packages/vitepress-plugin-moss/demo-site getting-started.md and
  index.md JS install commands and imports -> @moss-dev/moss-web.
- Generated TypeDoc reference docs under
  packages/vitepress-plugin-moss/demo-site/documentation/docs/reference/:
  - js/*: renamed @moss-dev/moss -> @moss-dev/moss-web and dropped the
    inherited v1.0.0-beta.7 version suffix down to v1.0.0 (moss-web has
    no betas published)
  - python/*: dropped the inherited v1.0.0b16 -> v1.0.0 (moss@1.0.0 is
    the only version on PyPI; same flag from @HarshaNalluru on
    reference/python/classes/MossClient.md)

Verification:
- `npm install` + `npx tsc --noEmit` on apps/next-js: clean
- `pnpm install --lockfile-only` on packages/vitepress-plugin-moss:
  clean; lock now has 3 @moss-dev/moss-web entries and zero non-hyphen
  @moss-dev/moss entries

All other @moss-dev/moss references in the repo (SDK source, Node demo
apps like moss-bun, docker/javascript, the Vercel SDK, test fixtures,
moss-live-labs TypeScript examples, root README code snippet, etc.)
are Node contexts and stay on @moss-dev/moss — those are not browser
usage.
Re-ran `npm install` in apps/next-js and `pnpm install --lockfile-only`
in packages/vitepress-plugin-moss after rebasing onto main. Main
picked up dependabot bumps (#159) and the new examples/javascript-web/
directory (#162) while this branch was open; this commit makes both
lockfiles consistent with the current package.json deps.
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 1 new potential issue.

View 5 additional findings in Devin Review.

Open in Devin Review

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot Apr 21, 2026

Choose a reason for hiding this comment

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

🚩 Pre-existing: Search.vue passes raw number instead of QueryOptions to query()

At Search.vue:321-322, the code reads topk as a number and passes it directly as the third argument to mossClient.value.query(). All @moss-dev/moss-web examples (e.g., examples/javascript-web/load_and_query_sample.ts:54) pass { topK: 3 } as a QueryOptions object. Additionally, the option is read as (options.value as any).topk (lowercase k) while types.ts:9 defines it as topK (camelCase), so the config value would never be found and always fall back to 20. This is a pre-existing issue not introduced by this PR, but the import switch to @moss-dev/moss-web makes it worth noting in case the new SDK is stricter about argument types.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

The @moss-dev/moss-web SDK uses `timeTakenMs` on SearchResult, not the
old `timeTakenInMs`. Earlier in this PR I updated the source in
actions.ts but missed the matching property in the test mock, so the
'returns success with docs and timeTaken' case read `undefined` from
the mock and CI failed. Mock now produces `timeTakenMs: 42`,
test passes locally (3/3).
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 2 new potential issues.

View 3 additional findings in Devin Review.

Open in Devin Review

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot Apr 21, 2026

Choose a reason for hiding this comment

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

🚩 Two distinct JS SDK packages documented differently across pages

The getting-started guide (getting-started.md:31) tells users to npm install @moss-dev/moss-web, while the JS SDK reference (reference/js/README.md:24) says npm install @moss-dev/moss. These are intentionally different packages: @moss-dev/moss-web is the browser/WASM SDK, and @moss-dev/moss is the Node.js SDK (confirmed by sdks/javascript/sdk/CHANGELOG.md:8 which states "Node-only: Dropped browser/WASM support"). However, the getting-started page links directly to the JS reference docs at line 149: [JavaScript SDK](/reference/js/README.md). A user following the getting-started flow would install the web SDK but then land on docs for the Node.js SDK, which has slightly different APIs (e.g., timeTakenInMs vs timeTakenMs). This could cause confusion.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

Devin correctly flagged that my previous commit was internally
inconsistent: the headers of `reference/js/**/*.md` got renamed to
`@moss-dev/moss-web` but the underlying interface definitions (e.g.
SearchResult.timeTakenInMs) still come from the Node SDK's typedoc
output. The two packages have slightly different APIs — moss-web uses
timeTakenMs, moss (Node) uses timeTakenInMs — so mislabelled headers
would send users reading property tables down the wrong path.

The correct target is the package the typedoc actually ran against,
which is `@moss-dev/moss` (Node). Pinning the reference docs to
`@moss-dev/moss v1.0.1` (the current latest on npm, no beta) keeps
the header consistent with the content.

Separately, @HarshaNalluru's guidance that `apps/next-js` and
`packages/vitepress-plugin-moss/Search.vue` should use
`@moss-dev/moss-web` is unchanged — those files stay on moss-web
(the browser/WASM runtime).
@ashvathsureshkumar
Copy link
Copy Markdown
Contributor Author

Round 3 fixes — thanks @devin-ai-integration:

Devin (2026-04-21T01:42, SearchResult.md:22 and globals.md:1) — correct catch. My previous commit relabelled the generated JS ref-doc headers to @moss-dev/moss-web v1.0.0 but the underlying property tables (e.g. SearchResult.timeTakenInMs) come from the Node SDK's typedoc output. moss-web and moss have slightly different shapes (timeTakenMs vs timeTakenInMs), so mislabelled headers would send readers down the wrong path. Reverted the entire reference/js/** tree in 4c0f699 to @moss-dev/moss v1.0.1 (the actual latest Node package, no beta), so header and content match again.

App code unchanged: apps/next-js and packages/vitepress-plugin-moss/Search.vue remain on @moss-dev/moss-web per @HarshaNalluru's guidance. Those are browser/WASM contexts; the generated refs describe the Node SDK.

Devin (README.md:24 install/import mismatch) — resolved by the same revert. Install is now npm install @moss-dev/moss and the import on :30 matches.

Devin (Cargo.toml:18 @moss-dev/moss-core verification) — verified: @moss-dev/moss-core@0.9.2 is published on npm with pre-built platform binaries (moss-core-darwin-arm64, moss-core-linux-x64-gnu, moss-core-linux-arm64-gnu, moss-core-win32-x64-msvc).

Devin (Search.vue:322 query signature) — pre-existing pattern difference across Search.vue vs actions.ts, not introduced by this PR. Leaving untouched; flag it on a future API-consistency pass.

Devin (earlier actions.ts:36 timeTakenMs-is-wrong) — Devin self-resolved in the 01:42 pass after inspecting the new examples/javascript-web/ that landed on main (#162). @moss-dev/moss-web uses timeTakenMs (confirmed in that PR's sample code), so my rename of the actions.ts property + test mock was correct.

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

Devin Review found 0 new potential issues.

View 3 additional findings in Devin Review.

Open in Devin Review

<div className="hero-header-inner">
<img
src="/images/InferEdgeLogo_Dark_Icon.png"
src="/Moss-logo.jpeg"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I dont see the new named image in public/images

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.

5 participants