Pull Request Explorer

Summary

Add ability to rotate Personal Access Tokens

Enables users to regenerate PAT secrets without deleting and recreating tokens, improving security and user experience by instantly invalidating old sessions and providing a one‑time secret display.

Health Assessment

• • PR had a long cycle time but a quick first review; the change touches core authentication and session logic, making it high risk despite AI assistance and extensive testing.

Summary

Remove AttributeBasedAccessControl feature flag

Eliminates the ABAC feature flag, consolidating control to the AccessControlSettings.EnableAttributeBasedAccessControl config setting and simplifying authorization logic.

Health Assessment

• • The PR had a single, quick review but involved a large code change and a slow overall cycle time, indicating moderate risk for regression in access‑control behavior.

Summary

Adopt per-target .PHONY directives in server Makefiles

This PR refines the server build scripts to eliminate merge conflicts caused by a single monolithic .PHONY declaration, improving maintainability of the build process.

Health Assessment

• • The PR was merged within 1.7 hours with a single review and no rework, indicating a straightforward change with minimal risk.

Summary

Bucket server test-timing cache by date to cut cache churn

Updates the server-test-merge CI workflow to use a date-based cache key, reducing duplicate cache creation and improving CI efficiency.

Health Assessment

• • Fast review and minimal changes indicate low complexity and quick approval.

Summary

Automated cherry pick of e2e cache config

Updates CI configuration to move e2e npm registry cache out of node-cache namespace, improving cache isolation and reducing test failures.

Health Assessment

• • Quick merge with minimal changes, indicating low risk and high confidence in the change.

Summary

Add daily cache warming workflows

Adds scheduled CI jobs to warm node and npm caches, improving build performance and reliability.

Health Assessment

• • Fast cycle time and minimal review rounds indicate a straightforward CI improvement with low risk.

Summary

Health Assessment

Summary

Fix broken links and tab rendering in docs

Improves documentation reliability by correcting internal links, converting legacy tab syntax, and restoring missing images, ensuring accurate navigation and rendering for users.

Health Assessment

• • Multiple review rounds and extensive changes indicate significant rework; large documentation changes may impact build and navigation.

Summary

Cherry pick of markdown autolink fix

Fixes markdown autolink trimming bug, ensuring links are correctly parsed in messages, improving user experience.

Health Assessment

• • Merged within 30 minutes with minimal changes, indicating low complexity and high confidence.

Summary

Remove internal pages from develop docs

Deletes outdated internal documentation pages, reducing clutter and potential confusion for developers.

Health Assessment

• • Merged within 36 minutes with no review comments, indicating a straightforward documentation cleanup with minimal risk to application behavior.

Summary

Remove ExperimentalAuditSettingsSystemConsoleUI feature flag

The experimental Audit Logging configuration page is now permanently enabled for Enterprise-licensed servers. The feature flag gating has been removed, simplifying configuration and reducing maintenance.

Health Assessment

• • Quick turnaround with minimal changes and fast review, indicating low complexity and low risk.

Summary

Rename the daily e2e npm cache to avoid potential prefix collisions

Renames the e2e npm cache key to prevent accidental cross‑restore with the built‑in npm cache, ensuring CI reliability and reducing future cache contamination risk.

Health Assessment

• • Fast review and merge with minimal changes; low risk of cache miss; CI configuration updated to avoid future cache collisions.

Summary

Fix draft upload interruption handling

Ensures users can resend messages after interrupted uploads, improving reliability of message composition.

Health Assessment

• • The PR had a long cycle time with multiple reverts and merges, indicating significant rework, but the final change was a bugfix that improved user experience by clearing stale upload states.

Summary

Add email to SAML audit log

Adds the user's email to SAML authentication audit logs to aid compliance and security teams in correlating sign‑up events.

Health Assessment

• • The PR was merged within 36 minutes with no review comments or rework, indicating a straightforward change with minimal risk.

Summary

Fix broken admonitions and anchors in docs

Corrects formatting and cross‑references in product‑overview changelog pages, ensuring proper rendering and navigation.

Health Assessment

• • Merged within 2 hours after a single review, indicating a straightforward documentation fix with minimal risk to runtime behavior.

Summary

Fix hyphenated word search in PostgreSQL

Improves search accuracy for hyphenated terms like "t-shirt", ensuring users find relevant posts.

Health Assessment

• • PR had a quick initial review but took several days for final approvals, indicating moderate review friction. The change is small and low risk, improving search accuracy for hyphenated terms.

Summary

Fix broken docs images, links, and layout

Improves documentation reliability by restoring missing images, correcting internal links, and fixing layout issues, ensuring users can access guides without broken references.

Health Assessment

• • Quick 4‑hour cycle with minimal review friction; documentation‑only changes with moderate file scope and low risk to production functionality.

Summary

Add docs CD workflow

Automates building and deploying the Docusaurus documentation site to S3 and CloudFront on every master push, ensuring up-to-date docs for users.

Health Assessment

• • Quick review and minimal rework; AI-assisted code generation streamlined the workflow creation.

Summary

Migrate docs site to Docusaurus with CI

Enables streamlined documentation build and search, improving developer experience and reducing build failures.

Health Assessment

• • Fast review and merge with minimal friction, small scope, low risk to core product.

Summary

Notify token owners on expired PAT deletion

Users receive a system bot DM when their personal access token expires and is deleted, improving visibility and reducing silent failures. This complements pre‑expiry warnings and ensures owners are aware of token removal.

Health Assessment

• • The PR was reviewed quickly but required several commits to address nitpicks and refactor, indicating moderate review friction. The feature adds significant new notification logic, but the slow cycle time suggests scheduling or complexity challenges.

Summary

Add bot accounts pagination to release 11.7

Enables full bot list loading and searchability by paginating bot accounts, improving performance and user experience.

Health Assessment

• • Fast review and merge with minimal changes, indicating low complexity and high confidence.

Summary

Warn users before personal access token expiry

Adds proactive notifications to users about expiring personal access tokens, preventing silent failures of integrations and improving reliability of token‑backed scripts and bots.

Health Assessment

• • The PR required extensive rework after the initial review, with many commits and multiple approval rounds, indicating significant review friction and complexity. The long cycle time (over 11 days) and large scope suggest a high‑risk change that required careful validation.

Summary

Bump Playwright v2 CI workers from 10 to 15

Increase Playwright worker count to reduce test duration, improving CI efficiency and reducing wait times for developers.

Health Assessment

• • Fast review and merge with minimal code changes; low risk to production and CI stability.

Summary

Remove WebSocketEventScope feature flag permanently

This change permanently enables typing and reaction event scoping, improving real-time performance and consistency without affecting user experience.

Health Assessment

• • Fast review and minimal rework indicate a straightforward refactor with low complexity.

Summary

Remove StreamlinedMarketplace feature flag

Disables the StreamlinedMarketplace feature flag, making the streamlined marketplace UI permanent and simplifying the marketplace modal. This removes legacy UI branches, updates tests, and ensures consistent user experience.

Health Assessment

• • The PR involved a substantial code change across backend and frontend, with rapid review and multiple commits to address feedback, indicating active iteration but low overall review friction.

Summary

Remove NotificationMonitoring feature flag

This PR removes the legacy NotificationMonitoring feature flag, permanently enabling notification metrics collection and simplifying configuration by relying solely on the existing MetricsSettings.EnableNotificationMetrics setting.

Health Assessment

• • Fast review and minimal rework indicate a smooth process; the cleanup of a legacy flag reduces maintenance overhead while keeping runtime behavior unchanged.

Summary

Migrate RFQA browser tests to Playwright

Migrates Rainforest QA browser tests to Playwright, expanding automated end‑to‑end coverage for search, multi‑team, direct messages, and group message flows, improving test reliability and reducing manual QA effort.

Health Assessment

• • Rapid review and single commit indicate smooth integration; extensive test migration expands coverage with minimal risk.

Summary

Add daily cache warming for CI jobs

Improves CI reliability by warming node and npm caches daily, reducing install times and ensuring read‑only restores for all CI workflows.

Health Assessment

• • Fast review cycle with minimal rework; medium scope of 10 workflow files; CI improvement with moderate risk due to broad impact on many jobs.

Summary

Migrate RFQA browser tests to Playwright

Migrates Rainforest QA browser tests to Playwright, improving test reliability and aligning with the new test framework.

Health Assessment

• • The PR involved a substantial migration of 20 E2E tests, requiring multiple commits and a CodeRabbit review, but the review cycle was quick and the merge was straightforward.

Summary

Restore ChannelBookmarks feature flag for mobile

Temporarily re‑enables the ChannelBookmarks flag so mobile clients can display bookmarks, preventing a UI break until the mobile app update is released.

Health Assessment

• • Quick, single‑file change with fast review and minimal comments, indicating low complexity and risk.

Summary

Forward session attribute updates to Desktop App

Enables the Desktop App to update a single session attribute field via websocket, eliminating full manifest refetches and improving performance.

Health Assessment

• • Fast review and minimal rework led to a quick merge with low risk.

Summary

Fix flaky TestSendMailWithEmbeddedFilesUsingConfig

Fixes intermittent test failures in mail sending by ensuring unique mailbox and proper retry logic, reducing CI flakiness.

Health Assessment

• • Fast review and minimal rework indicate a straightforward fix; the change is confined to a single test file and does not affect production code.

Summary

Cherry pick of scroll correction fix

Fixes scroll pop issue caused by delayed resize observer debounce, improving user experience when navigating channels or RHS.

Health Assessment

• • Quick cherry pick with minimal changes, fast review and merge, low risk.

Summary

Migrate RFQA browser tests to Playwright

Migrated the first batch of Rainforest QA browser tests into Playwright, improving test stability, maintainability, and reducing duplication of selectors and helpers.

Health Assessment

• • Large-scale test migration completed in under two days with AI-assisted review, minimal rework, and no reported regressions.

Summary

Update latest patch version to 11.9.1

Bumps the application and package versions to 11.9.1, ensuring consistency across the codebase and dependencies.

Health Assessment

• • Rapid merge with minimal review indicates low complexity and high confidence in version bump.

Summary

Update latest minor version to 11.10.0

This PR updates release metadata and package manifests to align with the new minor version, ensuring consistency across the codebase.

Health Assessment

• • Rapid merge with minimal review indicates low complexity and high confidence in the change.

Summary

Fix milestone lookup in docs-needed workflow

Ensures the docs-needed workflow correctly matches existing milestones, preventing duplicate milestone creation and improving automation reliability.

Health Assessment

• • Quick fix with immediate review and merge, minimal changes to a single workflow file, indicating low complexity and risk.

Summary

Improve license error for wrong service environment

Provides a specific error message when uploading a license signed for a different service environment, instead of a generic error.

Health Assessment

• • The PR had multiple review rounds and required several changes before being approved, indicating some complexity in the code changes.

Summary

mmctl: list and optionally auto-add missing team members on channel move

Adds client‑side detection of missing channel members when moving a channel, lists them, and introduces a --auto-add-users flag to automatically add those users to the destination team, improving the mmctl command experience.

Health Assessment

• • The PR was reviewed quickly (0.1h to first review) and merged after a single round of comments, but the change involved over 500 lines of Go code, adding a new flag and membership logic. AI assistance was used both to author and review the code, enabling a fast turnaround.

Summary

Fix plugin RPC teardown race on shutdown

Removes a race condition that caused plugins to receive spurious errors after shutdown, improving stability during restarts and upgrades.

Health Assessment

• • The PR required multiple commits after a single review and had a long cycle time, indicating significant complexity and a high risk of subtle timing regressions.

Summary

Fix flaky TestSendInviteEmails

Ensures reliable test execution by isolating the Inbucket mailbox for invitation emails, eliminating intermittent failures without impacting production code.

Health Assessment

• • Long cycle time due to scheduling, but the review was immediate and the change was minimal, resulting in low risk and quick resolution.

Summary

Restrict sidebar resizing to left mouse button

Ensures the sidebar divider only responds to left-click drags, reducing accidental resizes and aligning with user expectations.

Health Assessment

• • Quick review and merge within 23 hours, minimal changes, low risk.

Summary

Fix package-lock.json

Corrects random changes in the npm lock file to stabilize dependency resolution.

Health Assessment

• • Quick, single‑commit fix with no review friction, minimal scope, and low risk to production.

Summary

Cherry-pick plugin config preservation for release

Ensures plugin configurations persist during patching and fixes test syntax, maintaining stability for the upcoming release.

Health Assessment

• • Quick review and merge with minimal rework, indicating a straightforward bug fix that supports the release schedule.

Summary

Add mmctl channel users list command

Adds a CLI subcommand to list channel members with pagination and JSON output, improving admin visibility without direct DB access.

Health Assessment

• • AI-assisted release notes and comments streamlined the review, but the PR had a long cycle time due to scheduling; the new read‑only command poses minimal risk and adds valuable admin functionality.

Summary

Cherry pick plugin config preservation

Preserves individual plugin configurations during patch when absent, ensuring plugin stability across releases.

Health Assessment

• • Merged within 1.1 hours with no review, indicating an automated cherry pick of a previously merged bugfix.

Summary

Virtualize scheduled posts list to fix slowness

Improved performance of the Scheduled messages page by virtualizing the list, enabling quick load for users with many scheduled posts.

Health Assessment

• • The PR underwent extensive review and rework, with multiple AI-assisted commits and a long cycle time, indicating significant complexity and potential risk to UI performance.

Summary

Fix confusing empty state in access control job details modal

Ensures the job details modal displays a neutral message when no channels are affected and preserves the correct empty state when a search yields no results, improving clarity for administrators.

Health Assessment

• • The PR was reviewed quickly (0.5h) and required only a few commits, but the overall cycle time was long (308h), likely due to external scheduling rather than code complexity. The changes are minimal and well-tested, indicating low risk.

Summary

Add job create, show, and cancel commands to mmctl

Expands the mmctl CLI with create, show, and cancel job subcommands, enabling administrators to manage jobs directly from the command line.

Health Assessment

• • Rapid review and single commit indicate focused changes, but the large code additions and removal of client‑side validation introduce moderate risk for regressions.

Summary

Update latest patch version to 11.8.4

Automated patch version bump to 11.8.4 for the 11.8.x release line, ensuring consistency across backend and webapp dependencies.

Health Assessment

• • Fast merge with no review needed, typical for automated version bump PRs.