Skip to content

[e2e-v3.3.3] throwaway test — DO NOT MERGE#505

Merged
sfreudenthaler merged 1 commit into
mainfrom
e2e-test/ai-workflows-v3.3.3
Jul 3, 2026
Merged

[e2e-v3.3.3] throwaway test — DO NOT MERGE#505
sfreudenthaler merged 1 commit into
mainfrom
e2e-test/ai-workflows-v3.3.3

Conversation

@sfreudenthaler

Copy link
Copy Markdown
Member

Throwaway PR to e2e-verify dotCMS/ai-workflows v3.3.3 (PR #61: surface internal SDK errors + Sonnet 5/Fable pricing) before moving the floating v3 tag. Will close this and delete the branch once verified.

@sfreudenthaler
sfreudenthaler requested a review from a team as a code owner July 3, 2026 17:10
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@sfreudenthaler
sfreudenthaler force-pushed the e2e-test/ai-workflows-v3.3.3 branch from f2c3d98 to 2ee3738 Compare July 3, 2026 17:14
@claude

claude Bot commented Jul 3, 2026

Copy link
Copy Markdown

Claude finished @sfreudenthaler's task in 1m 27s —— View job


dotCMS Diff Analyst — PR #505

  • Read doc-object schema & rollback-unsafe categories
  • Fetch PR metadata & full diff
  • Check for linked issues / matching epic
  • Generate doc object (Output A)
  • Assess rollback safety (Output B)
  • Post results & apply label

Scope: one added file — .github/workflows/zz-e2e-test-v3.3.3.yml (+47/−0). No linked issues (closes/fixes/resolves) in the body; the dotcms-aios epics path returned 404, so no epic matched.


🟢 Rollback Safety: SAFE

Checked the diff against every category in ROLLBACK_UNSAFE_CATEGORIES.md (C-1…C-4, H-1…H-7, M-1…M-4). No match.

The only change is a new GitHub Actions workflow that calls the reusable claude-orchestrator.yml@v3.3.3. It is pure CI scaffolding:

  • No runonce/migration task → not C-1/C-3/C-4, H-1/H-2/H-3/H-6/H-7, M-1
  • No ESMapping* / putMapping() / reindex → not C-2
  • No new Field type → not H-4
  • No StoragePersistenceProvider change → not H-5
  • No REST/GraphQL response shape, *Bundler, or OSGi interface change → not M-2/M-3/M-4

Per the rule of thumb, N-1 starts, reads data, and serves requests unchanged — a .github/workflows/*.yml file has no bearing on the running application. It also carries the CI/CD label and is explicitly throwaway (DO NOT MERGE).

Label applied: AI: Safe To Rollback.


📄 Doc Object Draft (attached to merge commit post-merge)
---
commit: PLACEHOLDER_SHORT_SHA
title: "test: throwaway e2e workflow for ai-workflows v3.3.3 (not for merge)"
type: ci
module: "CI/CD — ai-workflows e2e test harness"
customer_visible: no
security_relevant: false
breaking_change: false
pr:
  primary: 505
release_notes:
  audience: skip
  priority: low
  reasoning: "Throwaway CI-only workflow added to e2e-verify a reusable-workflow release; no product surface and explicitly not for merge."
provenance:
  generator: doc-object-skill
  model: claude-opus-4-8-20251001
  prompt_version: v0.1
  generated_at: 2026-07-03T17:16:00Z
  source_pr: 505
  source_diff_sha: PLACEHOLDER_FULL_SHA
---

What changed

Adds a single throwaway GitHub Actions workflow, .github/workflows/zz-e2e-test-v3.3.3.yml, that calls the reusable dotCMS/ai-workflows/.github/workflows/claude-orchestrator.yml@v3.3.3 to end-to-end verify ai-workflows v3.3.3 before the floating v3 tag is moved. It defines two jobs, both gated on the PR title containing [e2e-v3.3.3]: a positive case that invokes global.anthropic.claude-sonnet-5 and expects a real review comment, and a negative case that invokes a bogus model id (global.anthropic.claude-does-not-exist-999) to confirm v3.3.3 now posts an explicit failure comment instead of silently swallowing the internal SDK error.

Risk / watch

Test-scaffolding only — no product code, no DB, ES, API, or storage surface is touched. The file is explicitly marked for deletion once the test PR is closed and should not be merged to main.

@sfreudenthaler
sfreudenthaler merged commit 2ee3738 into main Jul 3, 2026
30 of 31 checks passed
@sfreudenthaler
sfreudenthaler deleted the e2e-test/ai-workflows-v3.3.3 branch July 3, 2026 17:20
sfreudenthaler added a commit that referenced this pull request Jul 3, 2026
Cleanup — removes the scratch workflow added directly to main for
e2e-verifying dotCMS/ai-workflows v3.3.3 (see closed PRs #505/#506).
Both the happy path (Sonnet 5) and the error-surfacing fix (PR #61)
verified working; no longer needed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant