Migrate SolStatus onto Shared Project DNA #2
validate.yml
on: pull_request
Repository validation
44s
Annotations
3 errors and 11 warnings
|
Repository validation
Process completed with exit code 1.
|
|
tests/monitor-exec.workerd.test.ts:
tests/monitor-exec.workerd.test.ts#L5
TypeError: Cannot read properties of undefined (reading 'getCurrentSuite')
❯ tests/monitor-exec.workerd.test.ts:5:1
|
|
tests/cli-help.test.ts > solstatus CLI > prints Effect 4 unstable CLI help without an Effect 3 @effect/cli peer:
tests/cli-help.test.ts#L13
AssertionError: expected 'DESCRIPTION\n CLI wrapper for SolSta…' not to match /@effect\/cli/cli
- Expected:
/@effect\/cli/
+ Received:
"DESCRIPTION
CLI wrapper for SolStatus infrastructure. Uses Effect 4 `effect/unstable/cli` because `@effect/cli` still peers Effect 3.
USAGE
solstatus [flags]
FLAGS
--cloudflare-account-id string Cloudflare Account ID
--cloudflare-api-token string Cloudflare API Token
--secret-alchemy-passphrase string Alchemy Passphrase for state secrets
--better-auth-secret string Better Auth Secret for authentication
--stage string Deployment stage (default: dev)
--phase choice Phase to execute (up/deploy, destroy, read/plan, dev) (choices: destroy, up, read, deploy, plan, dev)
--quiet Run in quiet mode
--app-name string Application name (default: solstatus)
--fqdn string Fully qualified domain name (optional - if not provided, a worker URL will be generated)
GLOBAL FLAGS
--help, -h Show help information
--version, -v Show version information
--wizard Start wizard mode for a command
--completions <bash|zsh|fish|sh> Print shell completion script (choices: bash, zsh, fish, sh)
--log-level <all|trace|debug|info|warn|warning|error|fatal|none> Sets the minimum log level (choices: all, trace, debug, info, warn, warning, error, fatal, none)
"
❯ tests/cli-help.test.ts:13:24
|
|
Repository validation
Node.js 20 is deprecated. The following actions target Node.js 20 but are being forced to run on Node.js 24: actions/cache@v4, actions/checkout@v4. For more information see: https://github.blog/changelog/2025-09-19-deprecation-of-node-20-on-github-actions-runners/
|
|
anti-slop(require-safety-comment-for-type-assertion):
packages/app/src/components/latency-range-chart.tsx#L349
packages/app/src/components/latency-range-chart.tsx:349:29: This type assertion has no `SAFETY:` justification. State the checked invariant immediately before the assertion or its containing statement.
|
|
anti-slop(require-safety-comment-for-type-assertion):
packages/app/src/components/latency-range-chart.tsx#L348
packages/app/src/components/latency-range-chart.tsx:348:29: This type assertion has no `SAFETY:` justification. State the checked invariant immediately before the assertion or its containing statement.
|
|
react(purity):
packages/app/src/components/latency-range-chart.tsx#L391
packages/app/src/components/latency-range-chart.tsx:391:36: Cannot call impure function during render
|
|
anti-slop(require-safety-comment-for-type-assertion):
packages/infra/src/stack-options.ts#L11
packages/infra/src/stack-options.ts:11:10: This type assertion has no `SAFETY:` justification. State the checked invariant immediately before the assertion or its containing statement.
|
|
react(set-state-in-effect):
packages/app/src/hooks/use-mobile.ts#L14
packages/app/src/hooks/use-mobile.ts:14:5: Calling setState synchronously within an effect can trigger cascading renders
|
|
anti-slop(no-known-value-widening):
packages/app/src/components/refresh-progress-bar.tsx#L14
packages/app/src/components/refresh-progress-bar.tsx:14:57: The explicit open dictionary type on binding `intervalLabels` discards known type evidence. Keep inference, validate with `satisfies`, or use a named owner contract.
|
|
anti-slop(no-shape-in-symbol-names):
packages/app/src/components/uptime-chart.tsx#L640
packages/app/src/components/uptime-chart.tsx:640:42: Rename symbol "shape" for its domain role; "shape" describes structure rather than ownership.
|
|
anti-slop(require-safety-comment-for-type-assertion):
packages/app/src/components/uptime-chart.tsx#L470
packages/app/src/components/uptime-chart.tsx:470:44: This type assertion has no `SAFETY:` justification. State the checked invariant immediately before the assertion or its containing statement.
|
|
anti-slop(no-unknown-parameters):
packages/app/src/components/uptime-chart.tsx#L469
packages/app/src/components/uptime-chart.tsx:469:27: Parameter `props` leaves input unparsed. Accept a named domain type; run the expected schema or parser at the I/O boundary before calling this function.
|
|
react(purity):
packages/app/src/components/uptime-chart.tsx#L507
packages/app/src/components/uptime-chart.tsx:507:36: Cannot call impure function during render
|