Skip to content

Merge dev: CI fixes + CLAUDE.md guidelines#4

Merged
brianfunk merged 6 commits into
masterfrom
dev
Feb 1, 2026
Merged

Merge dev: CI fixes + CLAUDE.md guidelines#4
brianfunk merged 6 commits into
masterfrom
dev

Conversation

@brianfunk

Copy link
Copy Markdown
Owner

Summary

  • Fixed CI failures (rollup optional deps, capstring dependency)
  • Added "Working Style" section to CLAUDE.md
  • Bumped to v1.0.1

Changes

  • .github/workflows/ci.yml: Clean install to resolve platform-specific deps
  • package.json: Use GitHub reference for capstring, bump version
  • CLAUDE.md: Working style guidelines

Test plan

  • CI passing on dev
  • Review changes

🤖 Generated with Claude Code

brianfunk and others added 6 commits February 1, 2026 01:36
- Add AbortController, setTimeout, clearTimeout to ESLint globals
- Wrap yaml/csv case blocks with braces to fix no-case-declarations

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Change from local file path to GitHub reference so CI can install
the dependency. Once capstring is published to npm, update to use
the npm package instead.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Bump to v1.0.1

npm ci fails with rollup optional dependencies on Linux when lockfile
was generated on Mac. Using npm install resolves platform-specific deps.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Remove package-lock.json and node_modules before npm install to
ensure platform-specific optional dependencies are correctly resolved.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Synthesized from Claude Code team best practices:
- Planning before coding for one-shot implementations
- Self-improvement by updating CLAUDE.md when corrected
- Autonomous problem solving for bugs/CI failures
- Critical code review mindset
- Iterative refinement over hacky solutions
- Context efficiency via subagents

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Distilled to actionable principles:
- Think first, code second
- Own problems autonomously
- Verify work, don't assume
- Rethink when stuck, update CLAUDE.md when wrong
- Use subagents to stay focused

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@brianfunk brianfunk merged commit fe819a9 into master Feb 1, 2026
6 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