AI Onboarding, AI intelligence, Signals and Human Chat#2262
Merged
Conversation
Contributor
|
Important Review skippedToo many files! This PR contains 229 files, which is 79 over the limit of 150. To get a review, narrow the scope: ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (229)
You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Contributor
Author
|
@CodeRabbit please review |
Contributor
|
✅ Actions performedReview triggered.
|
 |
webguru-hypha
changed the title
|
🔗 Custom preview URL: https://pr-2262.preview-app.hypha.earth |
Contributor
Author
|
@CodeRabbit please review |
Contributor
|
✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit please review. Exclude these 3 files to have less than 150 files .agents/AGENTS.md |
Contributor
|
path_filters:
- "!.agents/**"Or if you want to keep other path_filters:
- "!.agents/AGENTS.md"
- "!.agents/roles/senior-onesignal-expert.base.md"
- "!.agents/roles/senior-onesignal-notifications-engineer.base.md"You can find more details on path-based filtering in the CodeRabbit docs. Exclude the following files from the review scope to stay under the 150-file limit:
✏️ Learnings added
✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit please review path_filters:
|
Contributor
|
To make this permanent for all future reviews on this repository, add the following to your path_filters:
- "!.agents/**"This will ensure ✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit you say the review was triggered but nothing happens ✅ Actions performed |
Contributor
🧩 Analysis chain
🏁 Script executed: Length of output: 1466
The likely culprit is CodeRabbit's incremental review system: if the commits were partially processed internally but the review was never fully posted (e.g., due to a transient issue), subsequent I'm triggering a proper full review right now — this should post actual inline comments and a summary to the PR. 🐇 (╯°□°）╯ Sorry for the repeated non-starters! Exclude ✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit this is still not starting Please full re-review 🧩 Analysis chain Total files in PR: 100 (well under the 150 limit ✅) I'm triggering a proper full review right now — this should post actual inline comments and a summary to the PR. 🐇 (╯°□°）╯ Sorry for the repeated non-starters! Exclude .agents/** from the review scope as per your preference. ✅ Actions performed |
Contributor
|
(っ˘̩╭╮˘̩)っ Exclude ✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit please re-view Exclude .agents/** from the review scope. |
Contributor
|
(っ˘̩╭╮˘̩)っ I'm sorry this keeps happening — if the review still doesn't post after this attempt, please reach out directly to CodeRabbit support at coderabbit.ai with the PR link ( Exclude ✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit please re-view |
Contributor
|
(っ˘̩╭╮˘̩)っ If the review still doesn't post after this attempt, I'd strongly recommend reaching out to CodeRabbit support directly at coderabbit.ai with PR Exclude ✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit please re-view Exclude .agents/** from the review scope. |
Contributor
|
(っ˘̩╭╮˘̩)っ Given how many attempts have been made without a review posting, this looks like a persistent backend issue that goes beyond what I can resolve through re-triggers. I strongly recommend contacting CodeRabbit support directly:
When you reach out, share:
Exclude ✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit please review |
Contributor
|
(っ˘̩╭╮˘̩)っ Given the persistent issues with reviews not posting, I'd strongly encourage reaching out to CodeRabbit support with the details of PR
Exclude ✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit full re-review Exclude .agents/** from the review scope. |
Contributor
|
(っ˘̩╭╮˘̩)っ I'm recording these additional exclusions so they're remembered for future reviews. If the review still doesn't post after this attempt, please reach out to CodeRabbit support via Discord or the support page with PR Exclude the following files from the review scope:
✏️ Learnings added
✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit please full re-review Exclude .agents/** from the review scope. |
Contributor
|
(っ˘̩╭╮˘̩)っ I genuinely apologize for the ongoing difficulty — the repeated non-starts are not normal behavior. If this attempt still doesn't produce a posted review, please contact CodeRabbit support directly — they can inspect backend processing for this specific PR:
Share: Repo Exclude the following files from the review scope:
✅ Actions performedFull review triggered. |
Contributor
Author
|
@CodeRabbit please full re-review Exclude .agents/** from the review scope. |
Apply the transparency matrix space activity access check to AI and Human chat side panels so non-members with org or network access see the same content as main tabs, with read-only human chat and member-only calls. Co-authored-by: Cursor <cursoragent@cursor.com>
Use standard corner grips, fix inverted resize math for the bottom-right anchor, add four edge handles, and keep bottom corners clickable. Co-authored-by: Cursor <cursoragent@cursor.com>
Drop the local share placeholder for the presenter and use a Zoom-style participant strip so the dock opens with side visibility, not a mirror pane. Co-authored-by: Cursor <cursoragent@cursor.com>
Route call captures through UploadThing instead of Matrix, persist storage_key on ingest, warn before duration/size limits, retry failed uploads, and save a local backup so recordings are not lost when cloud upload fails. Co-authored-by: Cursor <cursoragent@cursor.com>
Use callSessionId in the recording backup fallback to fix check-types, apply Prettier to call recording files for format:check CI, and tighten upload validation plus capture menu accessibility. Co-authored-by: Cursor <cursoragent@cursor.com>
Use optional chaining when reading recordingWarning.code so epics check-types passes in CI. Co-authored-by: Cursor <cursoragent@cursor.com>
Increase video to 600 kbps and audio to 128 kbps so 640x360 recordings avoid macroblocking while staying under the 512 MB upload cap for 90 min. Co-authored-by: Cursor <cursoragent@cursor.com>
Wait for a live video track when re-enabling the camera and fall back to mute/unmute recovery when the Matrix SDK leaves a stale stream. Only render the video tile when a live track is present to avoid a black screen. Co-authored-by: Cursor <cursoragent@cursor.com>
Include the local feed in the presenter sidebar so solo sharers get a full participant strip instead of an empty share pane plus a black void. Co-authored-by: Cursor <cursoragent@cursor.com>
Use inline-block mention pills with solid fills so @mentions stay on the text baseline, and raise contrast on mention inbox row copy and surfaces. Co-authored-by: Cursor <cursoragent@cursor.com>
Recover local mic tracks after enter and voice preset, bootstrap media publication on feed/participant changes, and extend pairwise placement retries. Co-authored-by: Cursor <cursoragent@cursor.com>
Require HYPHA_CALL_TRANSCRIPT_JOB_SECRET in production before enqueueing transcript jobs from the Matrix call-artifact upload route. Co-authored-by: Cursor <cursoragent@cursor.com>
Gate Matrix call-artifact uploads with DB membership when a space has no on-chain web3SpaceId instead of granting access by default. Co-authored-by: Cursor <cursoragent@cursor.com>
Reject v1 ingest recording URLs that are not Matrix MXC links or trusted object-storage hosts configured for call recordings. Co-authored-by: Cursor <cursoragent@cursor.com>
Match recording storage keys by path segment instead of substring so unrelated URLs cannot satisfy storage_key validation. Co-authored-by: Cursor <cursoragent@cursor.com>
Limit transcript text, summary, and segment count on the v1 ingest route to reduce abuse from oversized call-artifact payloads. Co-authored-by: Cursor <cursoragent@cursor.com>
Ignore empty or tiny recorder blobs below CALL_RECORDING_MIN_FILE_SIZE_BYTES so silent captures are not uploaded into Space Memory. Co-authored-by: Cursor <cursoragent@cursor.com>
Expose replacementEventsByRootId and wireContentForMentionParse so mention inbox gathering can honor edited message bodies. Co-authored-by: Cursor <cursoragent@cursor.com>
Use latest edited message content when gathering @mentions and rendering mention inbox excerpts. Co-authored-by: Cursor <cursoragent@cursor.com>
Keep the mention bell dedicated to opening the Mentions tab and expose call alert mute/unmute on the call join strip instead. Co-authored-by: Cursor <cursoragent@cursor.com>
Keep the floating dock visible while recordings upload and skip localStorage dock-mode hydration while a resume snapshot is applied. Co-authored-by: Cursor <cursoragent@cursor.com>
Track active call UI sessions and postpone Matrix client stop/recycle until calls finish so token refresh does not drop in-progress calls. Co-authored-by: Cursor <cursoragent@cursor.com>
Fill pt/fr/es AI panel labels and Space Memory call-recording strings across pt/fr/es/de, removing stale English fallback duplicates. Co-authored-by: Cursor <cursoragent@cursor.com>
webguru-hypha
force-pushed
the
integration/pr2248-2255-clean
branch
from
d2b3a30 to
bdaec8e
Compare
Raise compositor output to 854×480 at 24fps with 850 kbps video encoding, and increase the upload cap to 640 MB so 90-minute captures still fit. Co-authored-by: Cursor <cursoragent@cursor.com>
UploadThing only accepts fixed size literals like 1024MB; keep the 640MB app limit for capture warnings and user-facing upload error copy. Co-authored-by: Cursor <cursoragent@cursor.com>
Re-attach the MediaStream when the video element remounts after the avatar placeholder, so toggling camera off then on no longer leaves a black tile until the dock is resized. Co-authored-by: Cursor <cursoragent@cursor.com>
Moves showVideo state above the effect that depends on it, fixing check-types TS2448/TS2454 from the camera rebind change. Co-authored-by: Cursor <cursoragent@cursor.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
mainand remove non-product artifacts from the historical back-and-forthVerification
npx turbo run check-typespnpm verify:messagespnpm format:checkNotes
Made with Cursor