Skip to content

chore(ci): migrate Claude workflows to onesdata claude-code-action wrapper - #19

Open
mbernabeu wants to merge 2 commits into
masterfrom
chore/migrate-claude-action-to-onesdata-wrapper
Open

chore(ci): migrate Claude workflows to onesdata claude-code-action wrapper#19
mbernabeu wants to merge 2 commits into
masterfrom
chore/migrate-claude-action-to-onesdata-wrapper

Conversation

@mbernabeu

Copy link
Copy Markdown

Cambia las referencias directas a anthropics/claude-code-action@v1 por el wrapper de la org onesdata/ops_workflows/.github/actions/claude-code-action@master y añade otel_token para que las ejecuciones emitan telemetría OTLP a portal.ops.onestic.com.

Archivos modificados:

  • .github/workflows/util-claude-task.yml
  • .github/workflows/util-claude.yml

Por qué va separado del contenido: el Claude App rechaza autenticarse cuando el workflow del PR difiere del de la rama por defecto (medida de seguridad). Una vez este PR esté en master, los PRs futuros pasan la validación sin tocar nada. Patrón validado end-to-end en onesdata/atmos-mcp#11.

🤖 Generated with Claude Code

mbernabeu and others added 2 commits May 28, 2026 16:46
Replace direct references to `anthropics/claude-code-action@v1` with the
org wrapper `onesdata/ops_workflows/.github/actions/claude-code-action@master`
and pass `otel_token` so every Claude Code run in CI emits OTLP telemetry
to the portal-apps claude_analytics dashboard.

Files: util-claude-task.yml, util-claude.yml

Lands separately from any content PR so the first run can clear the
Claude App workflow-validation check.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Both util-claude workflows authenticated via `anthropic_api_key`, which
meant they were the only outliers in the org and missed the
auto-injected `otel_token` (the migration script anchored on
`claude_code_oauth_token`). Switch to the same OAuth token the rest of
the org uses so:

- Telemetry attribution is consistent (user.email = marketing@onestic.com)
- The wrapper's telemetry step actually fires
- No special-case auth path in this repo

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

1 participant