Commit Graph

23 Commits

Author SHA1 Message Date
Cole Medin
763e5b8244
CI fails now when unit tests for backend fail (#536)
* CI fails now when unit tests for backend fail

* Fixing up a couple unit tests
2025-08-30 12:52:34 -05:00
Cole Medin
9f22659f4c
Moving Dockerfiles to uv for package installation (#533)
* Moving Dockerfiles to uv for package installation

* Updating uv installation for CI
2025-08-30 11:33:11 -05:00
Rasmus Widing
b5e5cddc68 Remove claude-review-fork.yml workflow
The Claude Code Action is not compatible with reviewing PRs from forks.
It always attempts to checkout the PR branch which fails for security reasons.
2025-08-19 11:16:37 +03:00
Rasmus Widing
db142280e2 fix: Restructure fork review workflow to avoid PR branch checkout
- Create isolated review context directory to prevent PR detection
- Move diff to changes.patch file in review-context directory
- Add explicit REVIEW_INSTRUCTIONS.md file for guidance
- Use standard 'prompt' parameter instead of 'override_prompt'
- This approach should prevent Claude Action from auto-detecting PR context
2025-08-19 11:05:06 +03:00
Rasmus Widing
6a23a57b26 Override event context to prevent PR checkout
- Set GITHUB_EVENT_NAME to workflow_dispatch to avoid PR detection
- Use override_prompt instead of direct_prompt for better control
- Create wrapper script for debugging
- Explicitly tell Claude not to checkout code
2025-08-19 11:02:47 +03:00
Rasmus Widing
a00883ac5e Fix Claude checkout issue for forked PRs
- Add environment overrides to prevent PR branch checkout
- Add explicit github_token for authentication
- Add direct_prompt to guide Claude to use diff file
- Override GITHUB_REF and GITHUB_SHA to stay on base branch
2025-08-19 11:00:33 +03:00
Rasmus Widing
506cbc18c4 Simplify fork PR review to single workflow with pull_request_target
- Remove complex two-stage workflow approach
- Use pull_request_target with security safeguards
- Add first-time contributor check and approval requirement
- Never checkout PR code - only analyze diff
- Mirror full review format from main claude-review workflow
- Manual trigger via @claude-review-fork for maintainers
2025-08-19 10:57:25 +03:00
Rasmus Widing
ea42f7a2ac Remove invalid mode parameter and improve event context
- Remove invalid mode: review parameter
- Update event context to simulate issue_comment
- Add direct_prompt to guide Claude to review the diff
- Update instructions to use Read tool for pr-diff.patch
2025-08-19 10:47:57 +03:00
Rasmus Widing
933d2deb3b Fix Claude Code Action authentication and context issues
- Remove invalid pr_number parameter
- Add explicit github_token to fix OIDC failure in workflow_run
- Add mode: review for proper review mode
- Create fake event.json to provide PR context
- Set environment variables to simulate PR event
2025-08-19 10:43:47 +03:00
Rasmus Widing
e554f7b1ae Fix exec declaration error in Stage 2 workflow
- Replace async exec with execSync to avoid declaration issues
- Add proper error handling for artifact extraction
- Use childProcess module directly instead of destructuring
2025-08-19 10:39:50 +03:00
Rasmus Widing
e46c5f7b07 Simplify authorization for external PR workflows
- Move authorization check to Stage 1 job condition
- Remove complex authorization job from Stage 2
- Fix duplicate exec declaration error
- Add unauthorized user message handling in Stage 1
- Trust Stage 1's authorization in Stage 2
2025-08-19 10:34:50 +03:00
Rasmus Widing
8f96ea9044 Fix external PR workflow permissions and error handling
- Grant pull-requests write permission for comment posting
- Add try-catch error handling with continue-on-error
- Ensure workflow continues even if comment posting fails
2025-08-19 10:29:58 +03:00
Rasmus Widing
c79040ad4a Fix base branch checkout in Stage 2 workflow
- Extract PR base branch from artifact instead of using workflow branch
- Add step to switch to correct base branch after downloading PR info
- Use PR base branch for diff generation instead of workflow branch
2025-08-19 10:19:36 +03:00
Rasmus Widing
0bb97d8e26 Add documentation for external PR review workflows
- Explains the two-stage security model
- Provides usage instructions for contributors and maintainers
- Includes troubleshooting and security considerations
2025-08-19 10:09:59 +03:00
Rasmus Widing
fc97b4f5bd Add Stage 2 secure review workflow for external PRs
- Runs after Stage 1 via workflow_run trigger
- Has access to repository secrets
- Downloads PR artifact and performs review
- Maintains security by never checking out fork code
2025-08-19 10:09:45 +03:00
Rasmus Widing
d64745991b Add Stage 1 workflow for external PR info collection
- Collects PR information without requiring secrets
- Triggers on pull_request events and @claude-review-ext comments
- Uploads PR details as artifact for secure processing
2025-08-19 10:09:30 +03:00
Cole Medin
dab5c6b7d3
Update bug_report.yml
Changing Archon Alpha to Beta in the issue template
2025-08-18 09:13:15 -05:00
Cole Medin
c929042086 Speeding up CI 2025-08-13 18:59:02 -05:00
Cole Medin
e020e3353c Bringing in first version of CI 2025-08-13 18:49:15 -05:00
Cole Medin
59084036f6 The New Archon (Beta) - The Operating System for AI Coding Assistants! 2025-08-13 07:58:24 -05:00
Juan Miguel
5f93fede3d
Enable Dependabot & CI (#66)
* Create dependabot.yml

Currently watches for updates in github actions, and current iteration, present in the root folder. Commented expansion on how to maintain previous iterations addded.

* CI for local development

* CI for docker build

* Use matrix strategy on docker build

Docker version uses 3.12, so its interesting to ensure it properly works with this version

* Enable python 3.10 backporting
2025-03-19 08:22:11 -05:00
Cole Medin
3b430830b1 Small updates to the issue templates 2025-02-24 14:29:28 -06:00
Cole Medin
d4254b53c3 Updating documentation and creating issue templates. 2025-02-24 14:23:26 -06:00