Skip to content

fix(sailpoint): complete integration validation - #7345

Merged
waleedlatif1 merged 3 commits into
stagingfrom
codex/sailpoint-validation-followup
Sep 1, 2026
Merged

fix(sailpoint): complete integration validation#7345
waleedlatif1 merged 3 commits into
stagingfrom
codex/sailpoint-validation-followup

Conversation

@waleedlatif1

Copy link
Copy Markdown
Collaborator

Summary

  • align all SailPoint tools, request constraints, response metadata, pagination, retries, and file handling with the current official API contract
  • add account-selection, access-request-config, and entitlement-request-config operations with complete block and registry wiring
  • regenerate the 40-action integration documentation with a manual description and exact nested outputs
  • preserve existing non-SailPoint tool schema behavior with regression coverage

Type of Change

  • Bug fix

Testing

  • bun run lint
  • bun run type-check
  • bun run check:audits
  • 314 focused integration, block, registry, metadata, and canvas tests
  • 72 documentation generator tests

Checklist

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

@vercel

vercel Bot commented Sep 1, 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 Sep 1, 2026 7:36am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The latest commit updates SailPoint’s visual branding consistently across the application, documentation, block metadata, and deployment integration catalog.

  • Replaces the monochrome SailPoint SVG with the official multicolor mark.
  • Changes the SailPoint integration background color to white across generated and source metadata.

Confidence Score: 5/5

The latest commit appears safe to merge.

No blocking failure remains.

Important Files Changed

Filename Overview
apps/sim/components/icons.tsx Replaces the application’s monochrome SailPoint icon with a multicolor branded SVG.
apps/docs/components/icons.tsx Mirrors the SailPoint icon update in the documentation application.
apps/sim/blocks/blocks/sailpoint.ts Changes the SailPoint block background to white while preserving its operation wiring.
packages/deployment-config/src/integrations.json Aligns deployed SailPoint integration metadata with the white branded background.
apps/docs/content/docs/integrations/sailpoint.mdx Aligns the SailPoint documentation card color with the updated integration metadata.

Reviews (3): Last reviewed commit: "fix(sailpoint): update integration icon" | Re-trigger Greptile

@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 24 files

Heads up: you’re close to your included review allowance. Set a flex budget so reviews don’t pause.

Fix all with cubic | Re-trigger cubic

Comment thread apps/sim/lib/internal/sailpoint/schema.ts
Comment thread apps/sim/tools/sailpoint/outputs.ts
Comment thread apps/sim/tools/sailpoint/outputs.ts
@waleedlatif1
waleedlatif1 force-pushed the codex/sailpoint-validation-followup branch from 5e801b5 to 7970e21 Compare September 1, 2026 05:24
@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile

@waleedlatif1

Copy link
Copy Markdown
Collaborator Author

@greptile please review the latest commit 9726bb9

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