Skip to content

refactor(web): remove the half-used client facade - #16

Merged
TheGreatAxios merged 1 commit into
mainfrom
cl-11859-remove-the-half-used-client-facade-from-the-interface
Oct 31, 2026
Merged

TheGreatAxios merged 1 commit into
mainfrom
cl-11859-remove-the-half-used-client-facade-from-the-interface

Conversation

@TheGreatAxios

Copy link
Copy Markdown
Contributor

Closes CL-11859

What changed

  • Callers import api/* directly; client.ts is deleted. Its four uncalled methods go with it.
  • updateProject call sites call the one operation each passed (renameProject, shipProject, archiveProject/restoreProject); deleteProject was already archiveProject.
  • The projects list keeps its query key, data shape and single-flight read.
  • The remaining .js relative specifiers in the interface are .ts; a stale comment about a deploy-and-bind prelude that no longer exists is removed.

How it was verified

bun run lint, bun run typecheck, bun run format:check and bun test apps/web pass.

@linear-code

linear-code Bot commented Oct 31, 2026

Copy link
Copy Markdown

CL-11859

@corbits-triage

corbits-triage Bot commented Oct 31, 2026 •

Copy link
Copy Markdown

Triage: Pass

No changes needed from you. A maintainer will review this pull request.

Triage will approve, request review.

Changed since the last verdict (Fail at 7f8c01c): 2 new commits.

@corbits-triage corbits-triage 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.

Triage requests changes:

  • Conventions: added code matches a code pattern

@TheGreatAxios
TheGreatAxios force-pushed the cl-11859-remove-the-half-used-client-facade-from-the-interface branch from 279bfdd to 7f8c01c Compare October 31, 2026 14:56

@corbits-triage corbits-triage 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.

Triage requests changes:

  • Conventions: added code matches a code pattern

@TheGreatAxios
TheGreatAxios force-pushed the cl-11859-remove-the-half-used-client-facade-from-the-interface branch from 7f8c01c to 190c7a6 Compare October 31, 2026 15:00
@corbits-triage
corbits-triage Bot dismissed stale reviews from themself October 31, 2026 15:02

Nothing triage asked to change still fails, so this request no longer stands.

@TheGreatAxios
TheGreatAxios force-pushed the cl-11859-remove-the-half-used-client-facade-from-the-interface branch from 190c7a6 to 4a393c3 Compare October 31, 2026 15:03
@TheGreatAxios
TheGreatAxios merged commit 4a393c3 into main Oct 31, 2026
4 checks passed
@TheGreatAxios
TheGreatAxios deleted the cl-11859-remove-the-half-used-client-facade-from-the-interface branch October 31, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant