Skip to content

ci: add PR validation workflow + clear lint debt + doc fixups - #23

Merged
hstarorg merged 2 commits into
mainfrom
dev
Jun 8, 2026
Merged

hstarorg merged 2 commits into
mainfrom
dev

Conversation

@hstarorg

@hstarorg hstarorg commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #22 (which shipped as v0.9.0).

  • Add a CI workflow (.github/workflows/ci.yml) — runs on pull_request + pushes to main/dev. The only prior workflow fired on v* tags, so PRs reached main with zero automated checks. Jobs:
    • web-ui: pnpm install --frozen-lockfile → lint → test → build
    • api: go build → go vet → make test
  • Clear pre-existing lint debt so the new lint gate is green from the start (all in files untouched by feature work):
    • eslint: scope react-refresh/only-export-components off for the generated components/ui/** dir (shadcn cva() *Variants co-exports — allowConstantExport doesn't cover call expressions).
    • AddRegistryUrl: drop the unused catch binding.
    • test setup: move a stale eslint-disable onto the actual as any.
  • Docs: drop the now-deleted docker-compose.yaml references in the design doc (layout tree + env-vars source note).

Verification (local)

  • pnpm lint / pnpm test (23) / pnpm build — green.
  • go build ./... / go vet ./... / make test — green.
  • Both workflows validated with actionlint (incl. shellcheck).

Not in this PR

  • The CHANGELOG 0.8.0 backfill (0.8.0's release-time splice never landed; --latest won't backfill it). Best done on main via git cliff v0.7.3..v0.8.0.

🤖 Generated with Claude Code

hstarorg and others added 2 commits June 8, 2026 17:46
The only existing workflow fires on v* tags, so PRs and pushes reached
main with zero automated checks. Add a CI workflow running web-ui
lint/test/build + api build/vet/test on pull_request and pushes to
main/dev.

Also clears the pre-existing lint debt that was already red on dev so
the new lint gate is green from the start:
- eslint: scope react-refresh/only-export-components off for the
  generated components/ui/** dir (shadcn cva *Variants co-exports).
- AddRegistryUrl: drop the unused catch binding.
- test setup: move the stale eslint-disable onto the actual `as any`.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
docker-compose.yaml was removed (local build+run now lives in
docker-compose.dev.yaml via `make dev`); update the layout tree and the
env-vars source note to point at the dev/ghcr compose files instead.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@hstarorg
hstarorg merged commit 7902169 into main Jun 8, 2026
4 checks passed
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