Skip to content

Claude Action#37

Merged
JuanGalilea merged 5 commits into
masterfrom
claude-action
May 12, 2026
Merged

Claude Action#37
JuanGalilea merged 5 commits into
masterfrom
claude-action

Conversation

@JuanGalilea
Copy link
Copy Markdown
Contributor

@JuanGalilea JuanGalilea commented May 11, 2026

Base action for Claude to respond within issues and PRs.

It can do changes in branches and then provides a 1 click PR creation. Got rid of the zenhub stuff(check first commit) since it seems somewhat orthogonal to the claude stuff and assumed that claude actually created a PR when handling an issue.

This is completely based off of Vladimir Zahajsky (@v-zahajsky)'s work at amazon.

There is no way of testing it here since claude's action enforces the PR's action to match action in the default branch. So there is no way to use the new action within a PR.

Tested it at https://github.com/apify-store/test-repo

@JuanGalilea JuanGalilea self-assigned this May 11, 2026
JuanGalilea

This comment was marked as off-topic.

Copy link
Copy Markdown
Contributor

@Patai5 Lukáš Průša (Patai5) left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

YOLO 🚀

@v-zahajsky
Copy link
Copy Markdown

We'll be able to have this workflow somewhere and have just link to this in repositores? It would be good to be able edit this independently on repos

Comment thread .github/workflows/claude.yaml Outdated
Comment on lines +54 to +65
// Create label if it doesn't exist yet
try {
await github.rest.issues.createLabel({
owner: context.repo.owner,
repo: context.repo.repo,
name: 'claude:done',
color: '7057ff',
description: 'Already processed by Claude'
});
} catch (e) {
// Label already exists, ignore
}

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just safety check. For store teams - we will be able to add it to parent repo and Lukas's actor will sync lables across all sotre repos. For prof services, we dont have something like that yet. And dont to do it probably. So I would at least comment this out - or delete it, after we decide we want to use this label arcoss repositories - I would like to use it everywhere. It also check that workflow ran correctly.

@JuanGalilea
Copy link
Copy Markdown
Contributor Author

Vladimir Zahajsky (@v-zahajsky) PR and issues creation ready.
Also no check for creation of label but a try catch in case its not set up (just to be safe)

@JuanGalilea JuanGalilea reopened this May 12, 2026
@JuanGalilea
Copy link
Copy Markdown
Contributor Author

also, action has a bug where it randomly crashes.
Its linked to symlink resolution but it still happens if you dont have so idk whats going on.
Its a P1 in misanthropic's repo but its been open for a month now, maybe they should ask claude to fix it 😆

@JuanGalilea JuanGalilea merged commit 8e4c0f4 into master May 12, 2026
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.

3 participants