supabase / cli / 36974905653
67%
develop: 67%

LAST BUILD BRANCH: slim-bump/postgres-15
DEFAULT BRANCH: develop
Ran
Jobs 1
Files 79
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 66.737%. Remained the same
36974905653

push

github

web-flow 
chore(cli): delete dead shared/auth layer and inline single-caller pass-throughs (#6953)

## Summary

A shallow-module audit of `apps/cli` found two kinds of cleanup. Each
candidate was checked with the deletion test: remove the module and
paste its body into every caller. If the callers end up no longer, the
module was only passing work through.

**Dead code.** Knip did not flag these because each file's only importer
was its own colocated unit test.

- The superseded auth layer under `apps/cli/src/shared/auth/` (api,
crypto, credentials, platform-api, token, errors). Live replacements are
`src/auth/command-*.ts` and `src/command-internal/login-*.layer.ts`.
- `shared/config/cli-project-local-service-versions.{service,layer}.ts`,
plus the two matching die-placeholders in `run.ts` and the mock in
`tests/helpers/mocks.ts`.
- `shared/output/table.ts` and `shared/output/time.ts`.

**Pass-throughs inlined or merged.**

- `db-connection.layer.ts` was a two-line alias; the sql-pg layer now
exports `dbConnectionLayer` directly.
- `OutputFormatFlag` moves into `command-internal/global-flags.ts` with
the other globals.
- The region display table is hoisted once to
`command-internal/region.format.ts` and shared by backups and projects.
- `changedLinkedLocalFlags` is called directly by seed and storage
instead of through per-command aliases; its coverage moves to the
db-target-flags unit test.
- `tracing.service.ts`, `db-bootstrap/messages.ts`,
`standalone-analytics-config.layer.ts`, `supabaseHome()`,
`output/bold.ts`, `go-string.ts`, `report.layers.ts`, and
`prepare.options.ts` are inlined at their single caller.
- `packages/cli-test-helpers` exports `harness.ts` directly instead of
through a barrel.

**Follow-up cleanups in later commits.**

- The error-tag stability snapshot
(`src/shared/telemetry/__fixtures__/error-tags.txt`) drops the four tags
that belonged to the deleted modules: `ApiError`,
`InvalidLocalServiceVersionsStateError`, `InvalidTokenError`, and... (continued)

4097 of 6139 relevant lines covered (66.74%)

7.37 hits per line

Jobs
ID Job ID Ran Files Coverage
1 36974905653.1 79
66.74
GitHub Action Run
Source Files on build 36974905653