Commit Graph

51 Commits

Author SHA1 Message Date
Mark Wylde
45ee2dca55 Attempt to make this work 2025-05-31 01:10:59 +01:00
Mark Wylde
e2f737a753 Attempt to make this work 2025-05-31 01:05:34 +01:00
Mark Wylde
1c309c8d10 Attempt to make this work 2025-05-31 00:59:31 +01:00
Mark Wylde
5171232878 Attempt to make this work 2025-05-31 00:56:07 +01:00
Mark Wylde
ea134ca929 Attempt to make this work 2025-05-31 00:50:13 +01:00
Mark Wylde
72aa15ac4f Attempt to make this work 2025-05-31 00:44:27 +01:00
Mark Wylde
87a39e8fbc Attempt to make this work 2025-05-31 00:35:22 +01:00
Mark Wylde
44d513b712 Attempt to make this work 2025-05-31 00:16:57 +01:00
Mark Wylde
3afac506b2 Attempt to make this work 2025-05-31 00:13:14 +01:00
Mark Wylde
436046a0ff Attempt to make this work 2025-05-31 00:09:10 +01:00
Mark Wylde
ed04634119 Attempt to make this work 2025-05-30 23:59:23 +01:00
Mark Wylde
8de76049e1 Attempt to make this work 2025-05-30 23:55:03 +01:00
Mark Wylde
bbf8371776 Attempt to make this work 2025-05-30 22:52:37 +01:00
Mark Wylde
e1be245c51 Attempt to make this work 2025-05-30 22:46:03 +01:00
Mark Wylde
0bb118b1a2 Attempt to make this work 2025-05-30 22:44:19 +01:00
Mark Wylde
4b69e8485a Attempt to make this work 2025-05-30 22:41:48 +01:00
Mark Wylde
4b1c3d000d Attempt to make this work 2025-05-30 22:31:06 +01:00
Mark Wylde
b41b7ecd9f Attempt to make this work 2025-05-30 22:18:35 +01:00
Mark Wylde
11685fc8c1 Attempt to make this work 2025-05-30 22:11:20 +01:00
Mark Wylde
87c1a97c6e Attempt to make this work 2025-05-30 22:02:19 +01:00
Mark Wylde
7018095f9a Attempt to make this work 2025-05-30 21:54:22 +01:00
Mark Wylde
e079f18247 Attempt to make this work 2025-05-30 21:49:42 +01:00
Mark Wylde
c0d1a3fc4c Attempt to make this work 2025-05-30 21:47:12 +01:00
Mark Wylde
c77bb0e4b3 Attempt to make this work 2025-05-30 21:20:59 +01:00
Mark Wylde
01602be052 Attempt to make this work 2025-05-30 21:12:47 +01:00
Mark Wylde
f2f966c77e Attempt to make this work 2025-05-30 21:00:03 +01:00
Mark Wylde
80886e1c8e Attempt to make this work 2025-05-30 20:49:55 +01:00
Mark Wylde
e2d102aadd Attempt to make this work 2025-05-30 20:40:22 +01:00
Mark Wylde
c004bcdb83 Attempt to make this work 2025-05-30 20:37:47 +01:00
Mark Wylde
2f36d061b3 Attempt to make this work 2025-05-30 20:34:43 +01:00
Mark Wylde
9d64c62a2e Attempt to make this work 2025-05-30 20:32:40 +01:00
Mark Wylde
828076e411 Attempt to make this work 2025-05-30 20:29:15 +01:00
Mark Wylde
9986f4d1a3 Attempt to make this work 2025-05-30 20:25:16 +01:00
Mark Wylde
406208cf7a Attempt to make this work 2025-05-30 20:20:51 +01:00
Mark Wylde
6410e33591 Attempt to make this work 2025-05-30 20:17:34 +01:00
Mark Wylde
f598608bb4 Attempt to make this work 2025-05-30 20:10:31 +01:00
Mark Wylde
e474962b0d First attempt 2025-05-30 20:02:39 +01:00
Ashwin Bhat
8da47815ec docs: add comprehensive FAQ covering common gotchas and limitations (#92)
- Add FAQ.md with sections on triggering, authentication, capabilities, and troubleshooting
- Document key limitations including workflow access, PR creation, and CI results visibility
- Include workarounds for common issues like automated workflows and test result access
- Cover security considerations and best practices for safe usage

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

Co-authored-by: Claude <noreply@anthropic.com>
2025-05-29 16:45:44 -07:00
Lina Tawfik
35ad5fc467 Add enhanced text sanitization (#83)
* Add enhanced text sanitization

* Format code with prettier

* Refactor tests to remove redundancy and improve structure

- Remove redundant 'mixed input patterns' test from sanitizer.test.ts
- Consolidate integration tests into 2 focused real-world scenarios
- Add HTML comment stripping to sanitizeContent function
- Update test expectations to match sanitization behavior
- Maintain full coverage with fewer, more focused tests

* Fix prettier formatting

* Remove rendered.html from repository

* Remove test-markdown.json and update .gitignore

* Revert .gitignore changes
2025-05-29 16:35:50 -07:00
Ashwin Bhat
fcbdac91f2 feat: add base_branch input to specify source branch for new Claude branches (#72)
* feat: add base_branch input to specify source branch for new Claude branches

- Add base_branch input parameter to action.yml allowing users to specify which branch to use as source
- Update setupBranch function to accept and use the base branch parameter  
- Defaults to repository default branch if no base branch is specified
- Addresses issue #62 for better branch control

Co-authored-by: ashwin-ant <ashwin-ant@users.noreply.github.com>

* perf: optimize setupBranch to avoid unnecessary default branch fetch

Only fetch repository default branch when actually needed:
- Skip initial fetch when baseBranch is provided
- Fetch default branch at end only for return value and GitHub Actions output
- Eliminates unnecessary API call when users specify base branch

Co-authored-by: ashwin-ant <ashwin-ant@users.noreply.github.com>

* fix: properly handle base branch throughout the action workflow

- Fix TypeScript error where defaultBranch was used before being assigned
- Replace DEFAULT_BRANCH with BASE_BRANCH in subsequent workflow steps
- Update PR creation and branch comparison to use the actual base branch
- Ensure custom base_branch input is respected in all operations

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

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

* refactor: move BASE_BRANCH env reading into parseGitHubContext

- Move BASE_BRANCH environment variable reading into parseGitHubContext for consistency
- Update setupBranch to use context.inputs.baseBranch instead of process.env
- Fix test descriptions to correctly reference BASE_BRANCH instead of DEFAULT_BRANCH
- Update test environment setup to use BASE_BRANCH

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

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

---------

Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
Co-authored-by: ashwin-ant <ashwin-ant@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>
2025-05-29 10:08:00 -07:00
Ashwin Bhat
52efa5e498 feat: display detailed error messages when prepare step fails (#82)
* feat: display detailed error messages when prepare step fails

- Capture prepare step errors in action.yml (up to 2000 chars)
- Add error details to comment update with collapsible section
- Handle both prepare and Claude execution failures separately
- Add test coverage for error detail display

This helps users debug issues like git errors, permission problems,
and branch creation failures more easily.

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

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

* refactor: simplify error capture to show clean error messages only

- Remove complex shell script that captured full output logs
- Use core.setOutput in prepare.ts to pass clean error message directly
- Avoid exposing potentially sensitive information from logs
- Show only the actual error message (e.g. 'Failed to fetch issue data')

This provides cleaner, more readable error messages without the risk
of exposing sensitive information from debug logs.

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

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

* refactor: simplify error display to show clean error messages only

- Remove collapsible <details> section for error messages
- Display errors in simple code blocks since messages are now clean and short
- Makes error messages more direct and readable

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

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

---------

Co-authored-by: Claude <noreply@anthropic.com>
2025-05-29 09:58:52 -07:00
Ashwin Bhat
37c3c29341 feat: allow user override of hardcoded disallowed tools (#71)
* feat: allow user override of hardcoded disallowed tools

Allow users to override hardcoded disallowed tools (WebSearch, WebFetch) by including them in their allowed_tools configuration. This provides users with the ability to control tool access based on their security requirements.

Changes:
- Modified buildDisallowedToolsString() to accept allowedTools parameter
- Added logic to filter out hardcoded disallowed tools if present in allowed tools
- Updated function call site to pass allowedTools
- Added comprehensive test coverage for override behavior
- Maintains backward compatibility

Resolves #49

Co-authored-by: ashwin-ant <ashwin-ant@users.noreply.github.com>

* prettier

---------

Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
Co-authored-by: ashwin-ant <ashwin-ant@users.noreply.github.com>
2025-05-29 09:40:32 -07:00
Ashwin Bhat
0eb34ae441 Add shallow fetch to improve performance for large repositories (#53)
* Add shallow fetch to improve performance for large repositories

This change adds `--depth=1` to git fetch operations to perform shallow
fetches instead of full history downloads. This significantly reduces
checkout time for large repositories as reported in issue #52.

Changes:
- Line 55: Added --depth=1 to PR branch fetch
- Line 102: Added --depth=1 to new branch fetch

Fixes #52

Co-authored-by: ashwin-ant <ashwin-ant@users.noreply.github.com>

* fetch 50 commits for PRs

---------

Co-authored-by: claude[bot] <209825114+claude[bot]@users.noreply.github.com>
Co-authored-by: ashwin-ant <ashwin-ant@users.noreply.github.com>
2025-05-27 16:31:06 -07:00
Ashwin Bhat
21e17bd590 remove .DS_Store (#69) 2025-05-27 13:26:03 -07:00
Ashwin Bhat
3c6a85b54b Improve error messages for GitHub Action authentication failures (#50)
- Add helpful hint about workflow permissions when OIDC token is not found
- Include response body in app token exchange failure errors for better debugging

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

Co-authored-by: Claude <noreply@anthropic.com>
2025-05-25 18:43:54 -07:00
Lina Tawfik
5b025a2e43 Fix prettier formatting 2025-05-23 11:31:08 -07:00
Lina Tawfik
a29981fe38 Remove inline comments from code 2025-05-23 11:22:47 -07:00
Lina Tawfik
c60a8fb69b Fix MCP server undefined error and file path resolution
- Add error field to MCP error responses to fix 'undefined' errors
- Add REPO_DIR environment variable to fix file path resolution
- Use GITHUB_WORKSPACE for correct repository directory
- Simplify path processing logic in commit_files tool

This fixes the issue where mcp__github_file_ops__commit_files would fail
with 'Error calling tool commit_files: undefined' by ensuring error messages
are properly formatted and files are read from the correct directory.
2025-05-23 11:17:05 -07:00
Lina Tawfik
e67f992a13 Update to use model parameter in claude-code-base-action
This updates claude-code-action to pass the model parameter to claude-code-base-action using the new primary `model` parameter instead of the deprecated `anthropic_model`.

This change is made in conjunction with https://github.com/anthropics/claude-code-base-action/pull/4 which adds the `model` parameter to claude-code-base-action.

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

Co-Authored-By: Claude <noreply@anthropic.com>
2025-05-22 09:10:44 -07:00
Lina Tawfik
dd5e8c974a feat: strip HTML comments from GitHub content
- Add stripHtmlComments function to remove HTML comments from text
- Apply to all GitHub content (bodies, comments, reviews, triggers)
- Add comprehensive tests for comment stripping functionality
2025-05-21 13:23:32 -07:00
Lina Tawfik
f66f337f4e Initial commit 2025-05-19 08:32:32 -07:00