Skip to content

feat(integrations): add Jama Connect and Flow Engineering - #9723

Merged
waleedlatif1 merged 7 commits into
stagingfrom
codex/jama-flow-integrations
Nov 6, 2026
Merged

waleedlatif1 merged 7 commits into
stagingfrom
codex/jama-flow-integrations

Conversation

@waleedlatif1

@waleedlatif1 waleedlatif1 commented Nov 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Add Jama Connect and Flow Engineering workflow actions and selectors using saved credentials.
  • Add live Search with personal account verification, project scopes, bounded reads, and account-bound references.
  • Reuse the existing credential dialogs, account limits, identity checks and eligibility rules for member setup and reconnect.
  • Refresh generated Assistant schemas from their current sources.

Type of Change

  • New feature

Testing

Validated all 40 Jama and 23 Flow operations against vendor API definitions. Ran focused connector, credential, selector, and Search tests; 49 credential lifecycle, isolation and multiple-account cases against disposable PostgreSQL/Redis with both schema provisioning paths; root test (39,715 app tests passed), lint, type-check, test-type regression check, block registry check, and all 63 audits. All six new Jama/Flow synthetic Search-discovery cases passed using Opus 5.5; four existing Lucid/Notion cases still failed. Authenticated vendor trial testing remains pending.

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)

Companion: simstudioai/mothership#926

@vercel

vercel Bot commented Nov 6, 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 Nov 6, 2026 2:46am UTC

Request Review

@greptile-apps

greptile-apps Bot commented Nov 6, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The reviewed changes appear safe to merge.

Summary

Adds Jama Connect and Flow Engineering actions, selectors, and live Search through saved credentials.

  • Workflow builders can use Jama Connect and Flow Engineering actions with saved accounts.
  • Search can find current Jama and Flow records through member accounts.
  • Members can save, reconnect, and remove personal Jama and Flow Search accounts.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart LR
  A[Member saves credentials] --> B[Check provider identity and email]
  B --> C[Save private connection]
  C --> D[Search with saved destination]
  D --> E[Check project scope]
  E --> F[Return connection-bound references]
  F --> G[Recheck connection before reading]
Loading

Reviews (5) · Last reviewed commit: "fix(flow): strip fences from generated J..." · Reviewed by Greptile

Comment thread apps/sim/blocks/blocks/flow.ts Resolved
Comment thread apps/sim/tools/flow/set-value.ts Resolved

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

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

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/tools/jama/delete_item.ts Resolved
Comment thread apps/sim/blocks/blocks/flow.ts Resolved
Comment thread apps/sim/blocks/blocks/flow.ts Resolved
Comment thread apps/sim/blocks/blocks/jama.ts Resolved
Comment thread apps/sim/lib/jama/client.ts Resolved
@waleedlatif1
waleedlatif1 force-pushed the codex/jama-flow-integrations branch from c1a6a17 to b8c894a Compare November 6, 2026 00:56
@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 Nov 6, 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.

@github-actions github-actions Bot added the requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep label Nov 6, 2026
@github-actions

github-actions Bot commented Nov 6, 2026

Copy link
Copy Markdown

⚠️ Cross-repo companion check

One or more companion PRs aren't merged into staging yet. Merging this without them will leave copilot and sim out of sync — merge them in lockstep.

Comment thread apps/sim/lib/sim-search/live/flow.ts Resolved

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

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

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/blocks/blocks/flow.ts Resolved
Comment thread apps/sim/tools/flow/set-value.ts Resolved
Comment thread apps/sim/blocks/blocks/jama.ts Resolved
@waleedlatif1
waleedlatif1 force-pushed the codex/jama-flow-integrations branch from b8c894a to dbdfbf2 Compare November 6, 2026 01:57
@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 Nov 6, 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 195 files

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

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/blocks/blocks/flow.ts Resolved
Comment thread apps/sim/lib/sim-search/live/flow.ts Resolved
Comment thread apps/sim/blocks/blocks/flow.ts Resolved
Comment thread apps/sim/lib/sim-search/live/flow.ts Resolved
@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 Nov 6, 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 195 files

Confidence score: 5/5

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

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/blocks/blocks/flow.ts Resolved
@waleedlatif1
waleedlatif1 force-pushed the codex/jama-flow-integrations branch from 41f6cdc to 8b3338c Compare November 6, 2026 02:40
@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 Nov 6, 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 197 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 7ee2814 into staging Nov 6, 2026
92 checks passed
@waleedlatif1
waleedlatif1 deleted the codex/jama-flow-integrations branch November 6, 2026 03:38
waleedlatif1 added a commit that referenced this pull request Nov 6, 2026
* Revert "improvement(org): move the People page onto the new design system (#9720)"

This reverts commit 7625929.

* Revert "fix(org): drop the removed shortcuts slot from the landing test (#9715)"

This reverts commit f17f1b6.

* Revert "improvement(org): move the organization view onto the new design system (#9708)"

The redesigned org view ships all at once with #9687. The shared modules later
staging work builds on stay: member-integrations and the accounts key factory.
The legacy integration detail opens the saved-credential modal for Jama and Flow,
which #9723 added only to the new view.

* fix(org): keep staging's shared modules and revert only the redesigned views

Hooks, contracts, key factories and lib helpers stay at staging, and the legacy
view adapts to them. Jama and Flow reconnect through the saved-credential modal
and disconnect through Search. Exports only the new views used are unexported or
deleted, and the e2e script returns to the legacy view's selectors.

This branch was successfully deployed

1 active deployment
Preview — 8b3338c6 Deployed Nov 6, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

requires-mothership-merge Has a companion PR on the mothership/copilot side — merge in lockstep

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant