Skip to content

Export graphqlToPageEntity from internal.ts#518

Merged
KevinDavilaDotCMS merged 2 commits into
mainfrom
KevinDavilaDotCMS-patch-15
Jul 23, 2026
Merged

Export graphqlToPageEntity from internal.ts#518
KevinDavilaDotCMS merged 2 commits into
mainfrom
KevinDavilaDotCMS-patch-15

Conversation

@KevinDavilaDotCMS

@KevinDavilaDotCMS KevinDavilaDotCMS commented Jul 23, 2026

Copy link
Copy Markdown
Member

Test to trigger the action

fixes #515

@claude

claude Bot commented Jul 23, 2026

Copy link
Copy Markdown

Claude finished @KevinDavilaDotCMS's task in 44s —— View job


✅ Rollback-Safety Analysis

  • Read docs/core/ROLLBACK_UNSAFE_CATEGORIES.md
  • Get full PR diff (53d7d930...31a4b39c)
  • Analyze diff against every unsafe category
  • Post verdict / apply label

Verdict: Safe to RollbackAI: Safe To Rollback label applied.

Diff analyzed (core-web/libs/sdk/client/src/internal.ts):

+// This is the export. This comment is only to trigger a build.
 export { graphqlToPageEntity } from './lib/utils/graphql/transforms';

This is a single-line comment addition to a TypeScript re-export file in the frontend SDK. No database migrations, runonce tasks, Elasticsearch mappings, contentlet_as_json model changes, REST/GraphQL contract changes, OSGi interfaces, or VTL viewtool contracts are touched. It doesn't match any category in C-1 through M-4 — this is a pure, inert code comment with zero runtime behavior change.

@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown

🤖 dotBot Review (Bedrock)

Reviewed 1 file(s); 0 candidate(s) → 0 confirmed, 0 uncertain (unverified, kept for review).

✅ No issues found after verification.


us.deepseek.r1-v1:0 · Run: #30040368308 · tokens: in: 1393 · out: 624 · total: 2017 · calls: 1 · est. ~$0.005

@github-actions

Copy link
Copy Markdown

⚠️ AI review failed

Claude did not produce a review — the backend call errored before generating any output (provider: anthropic-bedrock, model: global.anthropic.claude-sonnet-5). This usually means the model has no Bedrock access grant in the target account, or the model ID is invalid — not a problem with this PR.

Run: #30036565955

@KevinDavilaDotCMS
KevinDavilaDotCMS merged commit 700ac30 into main Jul 23, 2026
35 checks passed
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.

Implement SDK release mechanism

3 participants