Skip to content

Rename Sharing to Pair Prompting and improve grant dialog UX#1444

Merged
mergify[bot] merged 2 commits intomainfrom
feature/rename-sharing-to-pair-prompting
Apr 23, 2026
Merged

Rename Sharing to Pair Prompting and improve grant dialog UX#1444
mergify[bot] merged 2 commits intomainfrom
feature/rename-sharing-to-pair-prompting

Conversation

@jeremyeder
Copy link
Copy Markdown
Contributor

@jeremyeder jeremyeder commented Apr 23, 2026

Summary

  • Rename the "Sharing" workspace section to "Pair Prompting" in the sidebar nav and card title
  • Remove the duplicate "Grant First Permission" button from the empty state (the header button is sufficient)
  • Add placeholder examples (e.g., platform-team / e.g., jdoe@example.com) and a help tooltip to the name input in the Grant Permission dialog

Test plan

  • Unit tests pass (sharing-section, sessions-sidebar)
  • E2E test assertions updated
  • Verified in local kind cluster — sidebar label, card title, empty state, and grant dialog all render correctly

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • Updates
    • The workspace sharing feature is now labeled as "Pair Prompting" throughout the interface, including navigation and workspace sections.
    • Improved the grant permission dialog with helpful tooltips and more informative placeholder text to guide users during permission setup.

…alog UX

Rename the "Sharing" workspace section to "Pair Prompting" across
sidebar nav, card title, and all tests. Remove the duplicate "Grant
First Permission" button from the empty state. Add placeholder examples
and a help tooltip to the name input in the Grant Permission dialog.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jeremyeder jeremyeder added the ambient-code:self-reviewed Self-reviewed by Ambient agent label Apr 23, 2026
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 23, 2026

Deploy Preview for cheerful-kitten-f556a0 canceled.

Name Link
🔨 Latest commit b7f58af
🔍 Latest deploy log https://app.netlify.com/projects/cheerful-kitten-f556a0/deploys/69ea84975bcc0e0008346e3c

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 23, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 7354bb79-0a6d-434c-8646-68d3ef437bc3

📥 Commits

Reviewing files that changed from the base of the PR and between 74900d0 and b7f58af.

📒 Files selected for processing (6)
  • components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/__tests__/sessions-sidebar.test.tsx
  • components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/sessions-sidebar.tsx
  • components/frontend/src/components/workspace-sections/__tests__/sharing-section.test.tsx
  • components/frontend/src/components/workspace-sections/sharing-section.tsx
  • e2e/cypress/e2e/screenshots.cy.ts
  • e2e/cypress/e2e/sessions.cy.ts

📝 Walkthrough

Walkthrough

Renames the "Sharing" sidebar navigation label to "Pair Prompting" across frontend components and all corresponding tests. Updates the grant permission dialog UX with tooltip-based field help and improved placeholder text. Removes conditional "Grant First Permission" button rendering in empty state.

Changes

Cohort / File(s) Summary
Sidebar Navigation
components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/sessions-sidebar.tsx, components/frontend/src/app/projects/[name]/sessions/[sessionName]/components/__tests__/sessions-sidebar.test.tsx
Replaces "Sharing" label with "Pair Prompting" in sidebar navigation item and updates corresponding test assertions.
Sharing Section Component
components/frontend/src/components/workspace-sections/sharing-section.tsx, components/frontend/src/components/workspace-sections/__tests__/sharing-section.test.tsx
Updates heading to "Pair Prompting", adds TooltipProvider wrapper and HelpCircle tooltip for subject name field (content varies by group/user type), standardizes grant permission placeholder to 'e.g., platform-team', removes conditional "Grant First Permission" button in empty state, and updates test expectations accordingly.
E2E Tests
e2e/cypress/e2e/screenshots.cy.ts, e2e/cypress/e2e/sessions.cy.ts
Updates Cypress selectors and assertions from "Sharing" to "Pair Prompting"; removes "Grant First Permission" button reference in dialog interaction tests.
🚥 Pre-merge checks | ✅ 6 | ❌ 2

❌ Failed checks (2 warnings)

Check name Status Explanation Resolution
Title check ⚠️ Warning Title does not follow Conventional Commits format; missing type prefix (feat/fix/chore/refactor/etc) required by guidelines. Reformat title to Conventional Commits: e.g., 'refactor(workspace): rename Sharing to Pair Prompting and improve grant dialog UX'
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (6 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Performance And Algorithmic Complexity ✅ Passed PR contains UI-focused changes with no performance or algorithmic complexity concerns. Proper memoization, O(n) checks only on submission, standard iteration patterns, and debounced search queries.
Security And Secret Handling ✅ Passed PR contains no security violations. UI rename from 'Sharing' to 'Pair Prompting' with UX improvements; all auth mechanisms unchanged and secure.
Kubernetes Resource Safety ✅ Passed PR modifies only frontend React components (.tsx files) in components/frontend/src/. No Kubernetes manifests, RBAC policies, deployment configs, or container specifications affected.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/rename-sharing-to-pair-prompting
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch feature/rename-sharing-to-pair-prompting

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

@mergify mergify Bot added the queued label Apr 23, 2026
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Apr 23, 2026

Merge Queue Status

  • Entered queue2026-04-23 20:55 UTC · Rule: default
  • Checks skipped · PR is already up-to-date
  • Merged2026-04-23 20:55 UTC · at b7f58af1ba168f2463724b5e256434ed45de49b2 · squash

This pull request spent 11 seconds in the queue, including 2 seconds running CI.

Required conditions to merge

@mergify mergify Bot merged commit 47e770f into main Apr 23, 2026
70 checks passed
@mergify mergify Bot deleted the feature/rename-sharing-to-pair-prompting branch April 23, 2026 20:55
@mergify mergify Bot removed the queued label Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ambient-code:self-reviewed Self-reviewed by Ambient agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant