6 Commits

Author SHA1 Message Date
Cheffromspace
12e4589169 Fix: Merge entrypoint scripts and fix auto-tagging tool permissions (#146)
* fix: merge entrypoint scripts and fix auto-tagging tool permissions

- Merged duplicate claudecode-entrypoint.sh and claudecode-tagging-entrypoint.sh scripts
- Added dynamic tool selection based on OPERATION_TYPE environment variable
- Fixed auto-tagging permissions to include required Bash(gh:*) commands
- Removed 95% code duplication between entrypoint scripts
- Simplified claudeService.ts to use unified entrypoint
- Auto-tagging now uses: Read,GitHub,Bash(gh issue edit:*),Bash(gh issue view:*),Bash(gh label list:*)
- General operations continue to use full tool set

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* fix: update Dockerfile to use unified entrypoint script

- Remove references to deleted claudecode-tagging-entrypoint.sh
- Update build process to use single unified entrypoint script

* fix: remove unnecessary async from promisify mock to fix lint error

* feat: add Husky pre-commit hooks with Prettier as primary formatter

- Added Husky for Git pre-commit hooks
- Configured eslint-config-prettier to avoid ESLint/Prettier conflicts
- Prettier handles all formatting, ESLint handles code quality only
- Pre-commit hooks: Prettier format, ESLint check, TypeScript check
- Updated documentation with pre-commit hook setup
- All code quality issues resolved

* feat: consolidate workflows and fix permission issues with clean Docker runners

- Replace 3 complex workflows with 2 lean ones (pull-request.yml, main.yml)
- Add Docker runner configuration for clean, isolated builds
- Remove file permission hacks - use ephemeral containers instead
- Split workload: GitHub-hosted for tests/security, self-hosted for Docker builds
- Add comprehensive pre-commit configuration for security
- Update documentation to be more pragmatic
- Fix credential file permissions and security audit

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* fix: allow Husky prepare script to fail in production builds

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

* fix: update CI badge to reference new main.yml workflow

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

---------

Co-authored-by: Claude <noreply@anthropic.com>
2025-05-31 20:53:58 -05:00
Jonathan
f42017f2a5 fix: resolve PR check failures for TypeScript and ESLint issues
- Remove unnecessary conditional checks in githubController.ts that caused TypeScript lint warnings
- Fix ESLint configuration to properly handle mixed JavaScript and TypeScript test files
- Update Jest configuration to remove deprecated isolatedModules option
- Add isolatedModules: true to tsconfig.json as recommended by ts-jest
- Ensure all tests pass and build succeeds

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-31 11:47:24 -05:00
Jonathan
223587a5aa fix: resolve all test failures and improve test quality
- Fix JSON parsing error handling in Express middleware test
- Remove brittle test case that relied on unrealistic sync throw behavior
- Update Jest config to handle ES modules from Octokit dependencies
- Align Docker image naming to use claudecode:latest consistently
- Add tsconfig.test.json for proper test TypeScript configuration
- Clean up duplicate and meaningless test cases for better maintainability

All tests now pass (344 passing, 27 skipped, 0 failing)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-29 12:33:20 -05:00
Jonathan Flatt
a38ed85924 feat: setup TypeScript infrastructure for Phase 1 migration
## Overview
Establishes comprehensive TypeScript infrastructure and tooling for
the claude-github-webhook project as specified in issue #101.

## Dependencies Added
- Core TypeScript: typescript, @types/node, @types/express, @types/body-parser
- Development: ts-node for dev execution
- ESLint: @typescript-eslint/parser, @typescript-eslint/eslint-plugin
- Testing: ts-jest, babel-jest for Jest TypeScript support

## Configuration Files
- tsconfig.json: Strict TypeScript config targeting ES2022/CommonJS
- eslint.config.js: Updated with TypeScript support and strict rules
- jest.config.js: Configured for both .js and .ts test files
- babel.config.js: Babel configuration for JavaScript transformation

## Build Scripts
- npm run build: Compile TypeScript to dist/
- npm run build Watch mode compilation
- npm run typecheck: Type checking without compilation
- npm run clean: Clean build artifacts
- npm run dev: Development with ts-node
- npm run dev Development with nodemon + ts-node

## Infrastructure Verified
 TypeScript compilation works
 ESLint supports TypeScript files
 Jest runs tests with TypeScript support
 All existing tests pass (67 tests, 2 skipped)
 Docker build process updated for TypeScript

## Documentation
- CLAUDE.md updated with TypeScript build commands and architecture
- Migration strategy documented (Phase 1: Infrastructure, Phase 2: Code conversion)
- TypeScript coding guidelines added

## Backward Compatibility
- Existing JavaScript files continue to work during transition
- Support for both .js and .ts files in tests and linting
- No breaking changes to existing functionality

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-27 19:35:54 -05:00
Jonathan Flatt
f36600aeca Fix linting errors across the codebase
- Remove unused variables and imports
- Add underscore prefix to intentionally unused caught errors
- Fix ESLint config to recognize underscore pattern for caught errors
- Update test mocks to use underscore prefix for unused parameters

All 12 linting errors resolved, only warnings remain.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-24 18:22:00 +00:00
Jonathan Flatt
2ab13ac736 feat: Update ESLint to v9 and fix linting issues
- Convert .eslintrc.js to eslint.config.js (ESLint v9 format)
- Add global definitions for fetch and URL
- Fix unused variables automatically via ESLint auto-fix
- Configure proper unused variable detection patterns
- All tests passing with good coverage (28 passed, 1 skipped)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-22 14:13:22 -05:00