fix: back navigation for integrations

[amhsirak]

closes #785

Summary by CodeRabbit

• Bug Fixes
  • Updated UI prop handling in RobotIntegrationPage to correctly pass onCancel and remove onArrowBack props.

[coderabbitai]

Walkthrough

RobotIntegrationPage updates UI prop wiring: it now passes onCancel={handleCancel} to RobotConfigPage in both render branches, and comments out onArrowBack={handleBack} in both places. The onBackToSelection prop remains commented. No other logic, state, or data flow changes.

Changes

Cohort / File(s)	Summary of Changes
RobotIntegrationPage UI prop wiring src/components/robot/pages/RobotIntegrationPage.tsx	Enabled passing onCancel to RobotConfigPage in both branches; commented out onArrowBack in both branches; left onBackToSelection commented; no other adjustments.

Sequence Diagram(s)

sequenceDiagram
  autonumber
  actor User
  participant RIP as RobotIntegrationPage
  participant RCP as RobotConfigPage

  Note over RCP,RIP: Current wiring
  User->>RCP: Click "Cancel"
  RCP-->>RIP: onCancel(handleCancel)
  RIP-->>User: Navigate/handle cancel

  Note over RCP: Arrow back not wired
  User->>RCP: Click "Arrow Back"
  RCP-->>User: No callback (inactive/ignored)

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• feat: pages ui revamp #754 — Previously altered RobotConfigPage’s onArrowBack/onCancel behavior; this PR reverts/adjusts the same prop wiring in RobotIntegrationPage.

Poem

I thump my foot—cancel’s back on track,
The arrow sleeps, no hop, no crack.
In wires neat, I twitch with glee,
A tidy path for you and me.
(·˵•‿•˵)っ🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	The title "fix: back navigation for integrations" succinctly and accurately captures the primary intent of the changeset, which adjusts back/navigation-related props in RobotIntegrationPage (onCancel now passed, onArrowBack commented out, onBackToSelection unchanged). It is concise, specific to the navigation fix, and clear enough for a teammate scanning history to understand the main change.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-integrations-back

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 02f09e0 and 6edddbe.

📒 Files selected for processing (1)

• src/components/robot/pages/RobotIntegrationPage.tsx (1 hunks)

🔇 Additional comments (1)

src/components/robot/pages/RobotIntegrationPage.tsx (1)

697-703: Incorrect — back navigation already wired; no change required.

RobotConfigPage's header arrow uses onArrowBack ? onArrowBack : handleBack, and handleBack calls onCancel() when provided. In the selection branch RobotIntegrationPage passes onCancel={handleCancel} (src/components/robot/pages/RobotIntegrationPage.tsx ~695–703), so the top-left arrow already invokes handleCancel; hiding the footer Cancel button (showCancelButton={false}) is intentional.

Likely an incorrect or invalid review comment.

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}