Skip to content

feat(cloudflare): expand API coverage and validate operations - #9194

Merged
waleedlatif1 merged 5 commits into
stagingfrom
codex/cloudflare-api-coverage
Oct 18, 2026
Merged

waleedlatif1 merged 5 commits into
stagingfrom
codex/cloudflare-api-coverage

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Oct 18, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Expand Cloudflare from 48 to 76 operations, adding Access groups and token lifecycle, Worker routes, tunnels, R2 policies, and missing resource operations.
  • Validate new and existing tools against the official API schema; expose missing inputs, nested outputs, and pagination metadata, and regenerate docs.
  • Fix resource-path validation, structured DNS records, explicit list clearing, response handling, and bounded zone-settings reads while preserving existing block input IDs.

Type of Change

  • Feature and bug fix

Testing

  • 84 focused Cloudflare tests and 35 docs-generator tests pass; new request-boundary and size-limit regressions failed before their fixes.
  • Root lint, type-check, all 60 audits, and block registry compatibility pass.
  • Full test suite runs in CI. API validation uses the official Cloudflare schema; live authenticated API calls were not performed.

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing (new tests pass the test-audit authoring gate)
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

@gitguardian

gitguardian Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
While these secrets were previously flagged, we no longer have a reference to the
specific commits where they were detected. Once a secret has been leaked into a git
repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@vercel

vercel Bot commented Oct 18, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
docs Ready Ready Preview Oct 18, 2026 10:02pm UTC

Request Review

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1
waleedlatif1 force-pushed the codex/cloudflare-api-coverage branch from bd313cb to b2ecf17 Compare October 18, 2026 21:07
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

The flagged value was an example from Cloudflare's public OpenAPI description for read_service_tokens_from_header, not an application credential. Replaced the example client ID and secret with explicit placeholders in the tools, block, and generated docs/metadata. The branch now contains only the corrected commit. Source: official Cloudflare API schema.

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 92 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/tools/cloudflare/update_access_group.ts
Comment thread apps/sim/tools/cloudflare/list_dns_records.ts
Comment thread apps/sim/tools/cloudflare/update_access_policy.ts
Comment thread apps/sim/tools/cloudflare/get_ruleset.ts Outdated
Comment thread apps/sim/tools/cloudflare/utils.ts
Comment thread apps/sim/tools/cloudflare/list_access_groups.ts Outdated
Comment thread apps/sim/tools/cloudflare/create_zone.ts Outdated
Comment thread apps/sim/tools/cloudflare/list_worker_routes.ts Outdated
Comment thread apps/sim/tools/cloudflare/utils.ts Outdated
Comment thread apps/sim/tools/cloudflare/list_access_applications.ts Outdated
@greptile-apps

greptile-apps Bot commented Oct 18, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The latest changes appear safe to merge; no actionable new issue was established.

Summary

Expands Cloudflare coverage from 48 to 76 operations and improves request checks, outputs, and pagination details.

  • Cloudflare workflows can now manage more resources and use richer API results.

Reviews (6) · Last reviewed commit: "fix(cloudflare): omit immutable ruleset ..." · Reviewed by Greptile

@greptile-apps

This comment has been minimized.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

Resolved: Access configuration gets dropped

Removed the duplicate advanced SaaS/target controls and aliases. Existing saasApp and targetCriteria inputs are now the sole source for both create and update requests. The two request-mapping regression cases failed before this change and pass now.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

Resolved: Zone updates send extra changes

Added a separate updateZoneType control with an empty Unchanged default. Creation keeps its full default; a pause-only update now sends only paused. The seeded-default regression failed before the fix and passes now.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

Resolved: Existing workflow outputs disappear

Restored the nullable legacy DNS zone_id, zone_name, and locked fields and service-token expires_at/last_seen_at fields. Values supplied by Cloudflare survive projection; omitted legacy values remain null. Regenerated tool and block metadata.

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

Resolved: Action tables promise wrong outputs

Cloudflare documentation now uses the evaluated per-tool output metadata, following the existing generator path for integrations with shared output constants. Regenerated the page and verified that Create Access Application contains only its application outputs. All 35 generator tests pass.

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 93 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/tools/cloudflare/delete_access_group.ts Outdated
Comment thread apps/sim/tools/cloudflare/list_access_applications.ts Outdated
Comment thread apps/sim/tools/cloudflare/update_zone_setting.ts Outdated
Comment thread apps/sim/tools/cloudflare/rotate_access_service_token.ts Outdated
Comment thread apps/sim/tools/cloudflare/list_r2_buckets.ts Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 93 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/tools/cloudflare/delete_worker_script.ts
Comment thread apps/sim/tools/cloudflare/update_ruleset.ts Outdated
Comment thread apps/sim/tools/cloudflare/list_r2_buckets.ts Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

All reported issues were addressed across 93 files

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/tools/cloudflare/update_ruleset.ts Outdated
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 18, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 93 files

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.

You've manually re-run cubic several times on this PR. Each manual re-review checks the full PR again and counts toward your usage quota. To preserve your usage limits, we recommend letting cubic automatically review new commits.

Turn on auto-fix | Re-trigger cubic

@waleedlatif1
waleedlatif1 merged commit 8c30945 into staging Oct 18, 2026
54 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/cloudflare-api-coverage branch October 18, 2026 22:12

This branch was successfully deployed

1 active deployment
Preview — d10c7ce4 Deployed Oct 18, 2026 by vercel[bot]
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