Skip to content

refactor(naming): drop legacy 'mothership' label#21

Merged
chitcommit merged 1 commit into
mainfrom
refactor/drop-mothership-naming
Jun 3, 2026
Merged

refactor(naming): drop legacy 'mothership' label#21
chitcommit merged 1 commit into
mainfrom
refactor/drop-mothership-naming

Conversation

@chitcommit

@chitcommit chitcommit commented Jun 3, 2026

Copy link
Copy Markdown
Member

Summary

Breaking change (minor)

  • /health response and X-ChittyOS-Service header now return chittyid instead of chittyid-mothership.
  • Downstream scan: one doc-only reference in CHITTYOS/chittyops/spec/daily-updates-orchestration-v0.5.md:172. No runtime matcher found.

Scope

  • Excluded archived dirs and *.bak / *.original files.

Test plan

  • node --check on touched JS / function modules
  • JSON parse on package.json, manifest.json, grafana dashboard
  • grep mothership returns zero hits in live files

Summary by CodeRabbit

  • Chores

    • Service rebranding: ChittyID Mothership is now called ChittyID
    • Updated all deployment configurations, worker names, endpoint URLs, and service identifiers
    • Refreshed service headers and registration metadata throughout the platform
  • Documentation

    • Updated production dashboard title and monitoring configurations to reflect new name
    • Refreshed UI branding in application footer and documentation pages

…l service name

Replace 'chittyid-mothership' with 'chittyid' and 'ChittyID Mothership' with
'ChittyID' across live files. The Cloudflare Worker has been named 'chittyid'
in wrangler.jsonc; this sweeps the lingering 'mothership' label out of code,
prose, config, monitoring, scripts, and the /health response body to conform
to the canonical Chitty<Word> service-naming rule.

Minor breaking change: /health and X-ChittyOS-Service header now return
'chittyid' instead of 'chittyid-mothership'. Backup/original files
(.bak/.original) and archived directories left untouched.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@chitcommit chitcommit merged commit bf07a1f into main Jun 3, 2026
0 of 2 checks passed
@coderabbitai

coderabbitai Bot commented Jun 3, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 72dc5cf1-f2b7-40f9-9a84-c7b47627a017

📥 Commits

Reviewing files that changed from the base of the PR and between ccabec4 and bb455f4.

📒 Files selected for processing (27)
  • .github/workflows/deploy.yml
  • CLAUDE.md
  • _worker.js
  • canon/api-routes-chittyid.js
  • canon/worker-chittyid.js
  • chitty-cli.ts
  • dashboard.html
  • functions/api/[[route]].js
  • manifest.json
  • monitoring/dashboards/grafana-chittyid.json
  • monitoring/metrics/prometheus.yml
  • scripts/deploy.sh
  • scripts/monitor-notion-sync.js
  • scripts/setup-env.sh
  • src/api/index.js
  • src/components/ChittyDashboard.jsx
  • src/config/index.js
  • src/middleware/pipeline-enforcer.js
  • src/middleware/request-interceptor.js
  • src/services/registry-client.js
  • src/services/topic-sync.js
  • src/workers/notion-sync-worker.js
  • test-advanced.sh
  • test-security.sh
  • tests/qa/pipeline-enforcement.test.js
  • worker.js
  • wrangler-pages.toml

📝 Walkthrough

Walkthrough

This PR renames the service identifier from chittyid-mothership to chittyid across the entire codebase, updating configuration files, API response headers and bodies, service registration metadata, deployment infrastructure, monitoring configurations, internal data fields, user-facing text, tests, and CLI integrations.

Changes

Service Identity Refactoring

Layer / File(s) Summary
Configuration and core service identity
manifest.json, wrangler-pages.toml, scripts/setup-env.sh
Manifest metadata, Pages project name, and wrangler.toml template generation updated to use chittyid service identifier.
API response headers and service metadata
src/middleware/pipeline-enforcer.js, src/middleware/request-interceptor.js, canon/api-routes-chittyid.js, functions/api/[[route]].js, worker.js, src/config/index.js, src/api/index.js
HTTP response header X-ChittyOS-Service and JSON response body fields across all handlers and middleware changed to advertise the new chittyid service identity.
Service registration and health discovery
src/services/registry-client.js
Service registration metadata, health check endpoint URLs, and HTTP headers updated to register and track the chittyid service identity.
Deployment and CI/CD infrastructure
.github/workflows/deploy.yml, scripts/deploy.sh
Deployment script constants and GitHub Actions workflow updated to target Cloudflare deployment under the chittyid project name.
Monitoring and observability
monitoring/metrics/prometheus.yml, monitoring/dashboards/grafana-chittyid.json
Prometheus job name and Grafana dashboard title updated to reflect the new service identity.
Internal data tracking and metrics
src/services/topic-sync.js, src/workers/notion-sync-worker.js
Topic creation metadata, message sender defaults, and metrics analytics indexes updated to use chittyid.
User-facing branding and documentation
CLAUDE.md, _worker.js, canon/api-routes-chittyid.js, canon/worker-chittyid.js, dashboard.html, functions/api/[[route]].js, src/components/ChittyDashboard.jsx
HTML page titles, headings, API documentation, and footer attribution text changed to display "ChittyID" instead of "ChittyID Mothership".
Test suite and validation
chitty-cli.ts, test-advanced.sh, test-security.sh, tests/qa/pipeline-enforcement.test.js, scripts/monitor-notion-sync.js
Test endpoint base URLs, test assertions, and CLI validation endpoints updated to target and verify the new chittyid service endpoint.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A hop and a skip, the Mothership's gone,
Now chittyid shines from dusk until dawn,
Through headers and configs, the renaming flows free,
One name change unites the whole symphony!

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/drop-mothership-naming

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 and usage tips.

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

Copy link
Copy Markdown

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: bb455f477b

ℹ️ 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".

version: '2.0.0',
description: 'Identity management system with hardened security pipeline',
endpoint: 'https://chittyid-mothership.chitty.workers.dev',
endpoint: 'https://chittyid.chitty.workers.dev',

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Advertise only a configured service endpoint

With the current worker config (wrangler.jsonc:9 sets workers_dev: false and the only route is id.chitty.cc/*), the renamed https://chittyid.chitty.workers.dev host is not served by this deployment. Registering that URL as the service endpoint makes registry consumers and later health updates probe an unreachable host after the rename; use the routed https://id.chitty.cc endpoint or enable workers.dev before advertising it.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed in #22 (commit 8236bb6): endpoint now reads from env.CHITTYID_SERVICE_URL / SERVICE_PUBLIC_URL, falling back to https://id.chitty.cc. No hardcoded workers.dev URL.

*/

const WORKER_URL = process.env.NOTION_SYNC_WORKER_URL || 'https://notion-sync.chittyid-mothership.workers.dev';
const WORKER_URL = process.env.NOTION_SYNC_WORKER_URL || 'https://notion-sync.chittyid.workers.dev';

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Keep the NotionSync workers.dev subdomain unchanged

When NOTION_SYNC_WORKER_URL is not set, the monitor now probes notion-sync.chittyid.workers.dev; in a workers.dev URL the segment after the worker name is the Cloudflare account subdomain, so renaming the ChittyID service label does not move the separate notion-sync worker from the existing chittyid-mothership subdomain. This default will make the health checks report the NotionSync worker as unreachable unless every environment overrides it.

Useful? React with 👍 / 👎.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Fixed in #22 (commit 8236bb6): NOTION_SYNC_WORKER_URL is now required; script fails fast with a clear error otherwise. No wrangler config in this repo deploys notion-sync, so guessing a canonical URL would be wrong (the worker may be dead code worth removing separately).

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