Skip to content

chore: format gate in CI, typecheck tests, release script, AGENTS.md - #16

Merged
crisner1978 merged 3 commits into
mainfrom
chore/retro-env
Oct 19, 2026
Merged

crisner1978 merged 3 commits into
mainfrom
chore/retro-env

Conversation

@crisner1978

Copy link
Copy Markdown
Collaborator

Retro follow-ups from shipping #15 / 0.1.25. Environment changes only; no runtime behavior changes.

Summary

 .github/workflows/ci.yml
+  - run: pnpm format:check        # repo was prettier-dirty on 30 files; swept once in 07c880a
 vitest.config.ts
+  typecheck: { enabled: true, include: ['test/**/*.test.ts'] }   # expectTypeOf assertions now compile
 package.json
+  "release": "pnpm version patch && git push --follow-tags && pnpm publish --access public"
 .gitignore
+  .claude/                        # tripped pnpm publish's clean-tree check
+.prettierignore                   # dist, docs, plans, lockfile, generated src/docs/data.ts
+AGENTS.md                         # where types come from, which spec is the contract, how to ship
 src/cli/commands.ts, src/mcp/server.ts
+  // prettier-ignore              # keeps the hand-aligned endpoint tables

Read commit-by-commit: d8dc606 (ignore + pins), 07c880a (mechanical sweep, no logic), ff2e4ab (the actual changes).

Evidence

  • Before: pnpm format:check → 30 files unclean; pnpm test ran type assertions without compiling them; publish needed --no-git-checks and a remembered --tag latest.
    After: pnpm format:check clean; pnpm test → 46 files / 308 tests incl. type pass, Type Errors: no errors; pnpm typecheck 0 errors. --tag latest is no longer needed once @auto.dev/sdk@1.0.0 is deprecated (done alongside this PR).

Merge Danger

Door: two-way. Revert restores everything; nothing persisted.

Blast Radius: CI

  • Open branches will fail format:check until rebased + pnpm format.
  • pnpm test runtime roughly doubles (typecheck pass per file); still ~seconds.

🤖 Generated with Claude Code

crisner1978 and others added 3 commits October 19, 2026 13:38
…bles

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
… script, AGENTS.md

- ci.yml runs pnpm format:check before typecheck + test.
- vitest typechecks test/**/*.test.ts so expectTypeOf assertions are real.
- pnpm release = version patch + push tags + publish (prepublishOnly builds).
- .claude/ gitignored (was tripping pnpm publish's clean-tree check).
- AGENTS.md: where types come from, which spec is the contract, how to ship.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@crisner1978
crisner1978 merged commit 1eba09d into main Oct 19, 2026
1 check passed
@crisner1978
crisner1978 deleted the chore/retro-env branch October 19, 2026 17:43
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