Skip to content

ci: Upgrade Claude action permission checks - #257

Merged
Paveltarno merged 1 commit into
mainfrom
fix/claude-action-v1
Aug 18, 2026
Merged

ci: Upgrade Claude action permission checks#257
Paveltarno merged 1 commit into
mainfrom
fix/claude-action-v1

Conversation

@Paveltarno

Copy link
Copy Markdown
Contributor

What?

Context: the interactive Claude workflows respond to @claude comments and spend the repository Anthropic API key. The pinned beta action lets any public GitHub user trigger a run.

This upgrades both workflows to v1.0.193, pinned at 9d7150bc8a3dae8149739a88019d192b579ad90c. v1 checks that the actor has write access before Claude starts.

How?

Migrates model and allowed_tools into claude_args, and direct_prompt to prompt, so the v1 upgrade keeps the existing model and tool restrictions instead of silently ignoring them.


🔗 Origin task in dev3: open in dev3 · dev3://task/9c1b6cf6-d979-4368-bd0e-6cb6aca8cc1c

Require write access before interactive Claude runs and migrate the workflows to the supported v1 prompt and CLI argument inputs.
@github-actions

Copy link
Copy Markdown

🚀 Package Preview Available!


Install this PR's preview build with npm:

npm i @base44-preview/sdk@0.8.43-pr.257.a72bddf

Prefer not to change any import paths? Install using npm alias so your code still imports @base44/sdk:

npm i "@base44/sdk@npm:@base44-preview/sdk@0.8.43-pr.257.a72bddf"

Or add it to your package.json dependencies:

{
  "dependencies": {
    "@base44/sdk": "npm:@base44-preview/sdk@0.8.43-pr.257.a72bddf"
  }
}

Preview published to npm registry — try new features instantly!

@ZachiNachshon ZachiNachshon left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@Paveltarno
Paveltarno merged commit 369a42d into main Aug 18, 2026
4 of 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.

2 participants