release: promote dev to prod — Homeport SDK 1.0.0#49
Conversation
Ships the repo's Voice-First 1.0 T1 train items and consolidates the delivered surfaces into README + docs/PRODUCT_ARCHITECTURE.md (or the repo's authoritative architecture doc). Removes stale roadmap and planning docs whose content is now current-state documentation. Highlights per repo are enumerated in workspace-level MASTER-RELEASE-ROADMAP.md (train completion record). Follow-ups and staged code are tracked in each repo's docs/ISSUES.md or the Staged-for-wiring section of the architecture doc. Validation: pnpm run validate green in the worktree. Signed-off-by: Tony <tony@nexartis.com>
feat(orchestration): Voice-First 1.0 T1 — delegation grant/revoke/check SDK methods
* chore(prod): release 1.2.0 * chore(prod): release 1.2.1 * chore(prod): release 1.2.2 --------- Signed-off-by: Pegasus HB3 <pegasus@nexartis.com> Co-authored-by: Pegasus HB3 <pegasus@nexartis.com> Co-authored-by: Lunavega888 <laura@nexartis.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Tony Rose <djsquircle+git@gmail.com> Co-authored-by: Jayme V <jaime@nexartis.com>
chore: sync prod 1.3.0 release-please bump into dev
* chore: baseline SDK foundation Signed-off-by: Pegasus HB3 <pegasus@nexartis.com> * fix: address baseline review comments Signed-off-by: Pegasus HB3 <pegasus@nexartis.com> --------- Signed-off-by: Pegasus HB3 <pegasus@nexartis.com> Co-authored-by: Pegasus HB3 <pegasus@nexartis.com>
BREAKING CHANGE: package renamed from @nexartis/nexartis-nanda-node-sdk; NnnClient->HomeportClient, NnnError->HomeportError, NnnErrorCode-> HomeportErrorCode; log prefix [homeport-sdk]; env NNN_*->HOMEPORT_*. - identity: package/README/community-health/typedoc/release-please -> @nexartis/homeport-sdk, docs https://homeport-sdk.nexartis.com, repo Nexartis/homeport-sdk; CHANGELOG reset (old archived) - src: full symbol rename; version drift fixed via scripts/gen-version.mjs + prebuild (SDK_VERSION generated from package.json); examples use https://homeport.example.com placeholder - tests: +3 namespace suites (agents/trust/orchestration incl. SSE + delegation): 162 -> 202 tests, all green; typecheck + build clean - docs site: workers homeport-sdk-docs[-dev] on homeport-sdk[-dev] .nexartis.com; landing rebranded (HomeportClient, CubiCube managed line); playwright smoke suite added (closes NNN-SDK-AUDIT-010) - NEW redirect-worker/: 301 nnn-sdk[-dev].nexartis.com -> new domains (6/6 unit tests); deploy-redirect.yml workflow - docs/REBRAND-RUNBOOK.md: operator checklist (repo rename, tag relocation, CF domain provisioning, publish gate, npm deprecate) Signed-off-by: Tony Rose <tony@nexartis.com>
Signed-off-by: Tony Rose <tony@nexartis.com>
…e, ignore test artifacts - pnpm-lock.yaml was missing @playwright/test + vitest (added to package.json in the rebrand but never locked) — frozen installs failed - landing e2e npm/yarn/bun tab locators hit strict-mode violations (npm matched pnpm); use exact matching - gitignore test-results/ and playwright-report/ Signed-off-by: Tony Rose <tony@nexartis.com>
- proper tablist/tabpanel structure for install tabs (aria-required-children, critical) - side-card headings + code-comment tokens to text-muted (color-contrast, serious) - footer columns h4 -> h3 (heading-order) Signed-off-by: Tony Rose <tony@nexartis.com>
…repo) Signed-off-by: Tony Rose <tony@nexartis.com>
…rotocol credit, drop trademark glyph Signed-off-by: Tony Rose <tony@nexartis.com>
…rect-worker dev env flag Signed-off-by: Tony Rose <tony@nexartis.com>
…lanned scope in ROADMAP Signed-off-by: Tony Rose <tony@nexartis.com>
Homeport SDK 1.0.0 — full rebrand from nexartis-nanda-node-sdk
🤖 Augment PR SummarySummary: Promotes the Homeport SDK rebrand and release setup from Changes:
Technical Notes: Release automation, docs deploy, and provenance publishing are updated to the new repository/package identity; legacy hostnames are preserved via redirects during the deprecation window. 🤖 Was this summary useful? React with 👍 or 👎 |
| working-directory: redirect-worker | ||
| run: pnpm install --frozen-lockfile || pnpm install | ||
|
|
||
| - name: Deploy to Cloudflare (dev) |
There was a problem hiding this comment.
In .github/workflows/deploy-redirect.yml line 61, the workflow deploys --env dev and then --env prod in the same job/environment; if the dev deploy fails (missing dev route/domain, transient API error), it will block the prod redirect deploy as well. Is it intended that production promotion depends on the dev deployment succeeding?
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
|
|
||
| - name: Install redirect-worker dependencies | ||
| working-directory: redirect-worker | ||
| run: pnpm install --frozen-lockfile || pnpm install |
There was a problem hiding this comment.
In .github/workflows/deploy-redirect.yml line 59, pnpm install --frozen-lockfile || pnpm install effectively disables lockfile enforcement and can change resolved versions at deploy time (including wrangler transitive behavior). That can make production deployments non-reproducible and harder to debug if a deploy starts failing without a source change.
Severity: medium
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
| @@ -1,89 +1,75 @@ | |||
| # @nexartis/nexartis-nanda-node-sdk | |||
| <p align="center"> | |||
| <a href="https://github.com/Nexartis/homeport"><img src="https://raw.githubusercontent.com/Nexartis/homeport-sdk/dev/.github/homeport-header.webp" alt="Homeport — the open-source, self-hostable NANDA node" width="800" /></a> | |||
There was a problem hiding this comment.
In README.md line 2, the header image is fetched from the dev branch (raw.githubusercontent.com/.../dev/...), so a future change to dev could break the image for npm/GitHub readers of the released package. Using a stable ref (default branch or a tag) helps keep the README render durable across releases.
Severity: low
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
| export default { | ||
| async fetch(request: Request, env: Env): Promise<Response> { | ||
| const url = new URL(request.url); | ||
| const target = env.TARGET.replace(/\/+$/, ""); |
There was a problem hiding this comment.
In redirect-worker/src/worker.ts line 20, env.TARGET.replace(...) will throw if TARGET is missing/empty at runtime, which would turn a misconfiguration into a 500 rather than a redirect. Given this worker is part of the deprecation window UX, it may be worth ensuring misconfig fails in a more controlled way.
Severity: low
🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: ac2f94bd8c
ℹ️ 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".
| baseUrl: 'https://nanda.nexartis.com', | ||
| apiKey: process.env.NNN_API_KEY, | ||
| const client = new HomeportClient({ | ||
| baseUrl: 'https://homeport.example.com', |
There was a problem hiding this comment.
Require a real base URL for runnable examples
The examples are still advertised as runnable with only HOMEPORT_API_KEY set, but this rebrand hard-codes the reserved placeholder https://homeport.example.com in the top-level examples. Anyone following examples/README.md and running them verbatim now sends every SDK call to a non-reachable placeholder instead of their Homeport node, so the examples fail before demonstrating the SDK; read a HOMEPORT_BASE_URL env var or document that this literal must be replaced.
Useful? React with 👍 / 👎.
| "deploy:dev": "wrangler deploy --env dev", | ||
| "deploy:prod": "wrangler deploy --env prod" | ||
| "deploy:prod": "wrangler deploy --env prod", | ||
| "test:e2e": "vitest run tests/redirect-worker.spec.ts && playwright test" |
There was a problem hiding this comment.
Stage docs before running docs-site E2E tests
This new test:e2e script launches Playwright against ./dist through playwright.config.ts, but the script never generates or stages that directory and typedoc-site/dist is gitignored. On a clean checkout following the new docs-site README (cd typedoc-site && pnpm run test:e2e), the landing and /api/ smoke tests will serve an empty/missing tree; make the script run the root docs staging step first or require a pre-staged dist explicitly.
Useful? React with 👍 / 👎.
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Files Reviewed (60+ files)
Notes: This PR promotes the already-merged Homeport rebrand (dev → prod) with no new logic. Changes verified are a consistent mechanical rename ( Reviewed by claude-sonnet-5 · Input: 54 · Output: 18.2K · Cached: 1.5M |
Promotes the merged Homeport rebrand (#47) to prod.
On merge this triggers:
Per workspace policy: human merges.