Skip to content

docs(hub): define backend code conventions - #102

Merged
nshki merged 2 commits into
build/hub/newfrom
copilot/hub-new-conventions
Oct 23, 2026
Merged

nshki merged 2 commits into
build/hub/newfrom
copilot/hub-new-conventions

Conversation

Copilot AI commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Why

The team needs shared code conventions before rebuilding Hub. This is the first PR for #101. It targets build/hub/new, not main.

What changes

Document repositories for database calls, integrations for API and SDK calls, utilities for general functions, and domains for business logic. Include the directory examples and rules for which layers may import others.

This PR changes only those conventions. The next PR covers the chosen tools, frontend setup, and removal of the old backend.

Copilot AI max review requested due to automatic review settings October 23, 2026 18:45
Copilot AI requested review from a team and nshki as code owners October 23, 2026 18:45

This comment was marked as duplicate.

Copilot AI changed the title docs(hub): define the TypeScript rebuild architecture docs(hub): document repositories, integrations, utilities, and domains Oct 23, 2026
Co-authored-by: Nishiki Liu <hello@nshki.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI changed the title docs(hub): document repositories, integrations, utilities, and domains docs(hub): define backend code conventions Oct 23, 2026
Copilot AI force-pushed the copilot/hub-new-conventions branch from bbdf1e3 to 3b51f1a Compare October 23, 2026 18:54
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot AI commented Oct 23, 2026

Copy link
Copy Markdown
Contributor Author

The next layer is #104. It sets up the new app and removes the old backend. Review this PR first, then #104; build/hub/new is still unchanged.

Copilot AI added this pull request to stack #105 October 23, 2026 19:22
@nshki
nshki merged commit ad26499 into build/hub/new Oct 23, 2026
28 checks passed
@nshki
nshki deleted the copilot/hub-new-conventions branch October 23, 2026 20:44
nshki added a commit that referenced this pull request Oct 23, 2026
<!-- Use a conventional commit-style title, such as "fix(hub): retain
prototype history". -->

## Why

<!-- Explain the problem and why this change matters in a short,
plain-English paragraph. Link the issue when applicable. -->

Give the team a clean starting point for rebuilding Hub without porting
product features yet. This follows #102 in the stack for #101. The stack
builds on `build/hub/new`, not `main`.

## What changes

<!-- Describe what becomes different for a first-time reader. Explain
necessary terms. Include accurate Mermaid diagrams when they clarify the
proposal, using before/after diagrams when helpful. Add screenshots for
meaningful UI changes. -->
<!-- Omit validation sections, checklists, and routine test/CI summaries
such as "CI is green" or "tests passed". Mention only known verification
gaps that affect the change, not disclaimers about drafting this
description. -->
<!-- For UI changes, prune obsolete and duplicate stories in the
affected catalogues as part of the change. Explain removed UI or
consolidated coverage where needed; see
hub/docs/storybook.md#ui-changes-and-pull-request-evidence. -->

Replace Rails and Superglue with a TypeScript Express API and React +
Vite app. The API has a health endpoint; the frontend shows the existing
logo and theme. Database, sign-in, and deployment choices remain open.

Keep the frontend components, styles, Storybook examples, and shared
canvas and Controls code for reuse. Remove the old backend, database
files, deployment setup, and live-update connection. The canvas keeps
its periodic refresh. Keep the presentation stories, move template
examples out of database seeds, and remove the deleted view from the
component-usage example.

Rewrite shared Ruby checks in TypeScript with function directories and
colocated Vitest tests. Update CI, local commands, and documentation for
the new setup.

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Nishiki Liu <hello@nshki.com>
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