iframe-proxy

mohitpubnub · 2026-05-22T08:07:36Z

No description provided.

…ases, include PUT type where needed. update body include cases when body present while making http requests.

… definition, updated pubnub-common file to have datasync apis support

coderabbitai · 2026-05-22T08:07:50Z

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (7)

dist/web/pubnub.js is excluded by !**/dist/**, !dist/** and included by none
dist/web/pubnub.min.js is excluded by !**/dist/**, !**/*.min.js, !dist/** and included by none
lib/types/index.d.ts is excluded by !lib/** and included by none
package.json is excluded by none and included by none
src/core/endpoints/data_sync/relationship/get-all.ts is excluded by none and included by none
src/core/pubnub-data-sync.ts is excluded by none and included by none
src/core/types/api/data-sync.ts is excluded by none and included by none

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: aa85bff6-bc0b-46d5-97cc-3af0f15674ee

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

📝 Walkthrough

Walkthrough

The CODEOWNERS file was updated to modify repository code ownership. The .github path owner list was changed to remove one owner and add another, while the README.md ownership remained unchanged.

Changes

Repository Code Ownership

Layer / File(s)	Summary
Update `.github` path owners `.github/CODEOWNERS`	The owner list for `.github` was modified: `@seba-aln` was removed and `@jguz-pubnub` was added, while `@mohitpubnub` and `@parfeon` remain. The README.md ownership line was unchanged.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title 'data sync' is vague and does not reflect the actual change, which is updating CODEOWNERS to change ownership from `@seba-aln` to `@jguz-pubnub`.	Use a more descriptive title that reflects the actual change, such as 'Update CODEOWNERS to change .github ownership' or similar.
Description check	❓ Inconclusive	No pull request description was provided by the author, making it impossible to assess relevance to the changeset.	Add a pull request description explaining the purpose of updating the CODEOWNERS file and the reason for changing ownership.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.

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

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch CLEN-3438

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

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

pubnub-ops-terraform · 2026-05-22T08:07:56Z

✅ Snyk checks have passed. No issues have been found so far.

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In @.github/CODEOWNERS:
- Line 1: The CODEOWNERS entry currently lists "jguz-pubnub" without the
required @ prefix; update the owner token on the same line so it reads
"`@jguz-pubnub`" (i.e., change the token "jguz-pubnub" to "`@jguz-pubnub`") to
ensure GitHub recognizes the user; locate the line containing "*        
`@mohitpubnub` `@parfeon` jguz-pubnub" and add the missing @ before jguz-pubnub.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f3bb076d-503f-4e28-b5b9-465bfb055e22

📥 Commits

Reviewing files that changed from the base of the PR and between 0ad7d97 and e82c07a.

⛔ Files ignored due to path filters (64)

dist/web/pubnub.js is excluded by !**/dist/**, !dist/** and included by none
dist/web/pubnub.min.js is excluded by !**/dist/**, !**/*.min.js, !dist/** and included by none
dist/web/pubnub.worker.js is excluded by !**/dist/**, !dist/** and included by none
dist/web/pubnub.worker.min.js is excluded by !**/dist/**, !**/*.min.js, !dist/** and included by none
lib/core/components/request.js is excluded by !lib/** and included by none
lib/core/constants/operations.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/entity/create.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/entity/get-all.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/entity/get.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/entity/patch.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/entity/remove.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/entity/update.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/relationship/create.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/relationship/get-all.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/relationship/get.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/relationship/patch.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/relationship/remove.js is excluded by !lib/** and included by none
lib/core/endpoints/data_sync/relationship/update.js is excluded by !lib/** and included by none
lib/core/pubnub-common.js is excluded by !lib/** and included by none
lib/core/pubnub-data-sync.js is excluded by !lib/** and included by none
lib/core/types/api/data-sync.js is excluded by !lib/** and included by none
lib/core/types/transport-request.js is excluded by !lib/** and included by none
lib/errors/pubnub-api-error.js is excluded by !lib/** and included by none
lib/transport/middleware.js is excluded by !lib/** and included by none
lib/types/index.d.ts is excluded by !lib/** and included by none
package-lock.json is excluded by !**/package-lock.json, !package-lock.json and included by none
src/core/components/request.ts is excluded by none and included by none
src/core/constants/operations.ts is excluded by none and included by none
src/core/endpoints/data_sync/channel/create.ts is excluded by none and included by none
src/core/endpoints/data_sync/channel/get-all.ts is excluded by none and included by none
src/core/endpoints/data_sync/channel/get.ts is excluded by none and included by none
src/core/endpoints/data_sync/channel/patch.ts is excluded by none and included by none
src/core/endpoints/data_sync/channel/remove.ts is excluded by none and included by none
src/core/endpoints/data_sync/channel/update.ts is excluded by none and included by none
src/core/endpoints/data_sync/entity/create.ts is excluded by none and included by none
src/core/endpoints/data_sync/entity/get-all.ts is excluded by none and included by none
src/core/endpoints/data_sync/entity/get.ts is excluded by none and included by none
src/core/endpoints/data_sync/entity/patch.ts is excluded by none and included by none
src/core/endpoints/data_sync/entity/remove.ts is excluded by none and included by none
src/core/endpoints/data_sync/entity/update.ts is excluded by none and included by none
src/core/endpoints/data_sync/membership/create.ts is excluded by none and included by none
src/core/endpoints/data_sync/membership/get-all.ts is excluded by none and included by none
src/core/endpoints/data_sync/membership/get.ts is excluded by none and included by none
src/core/endpoints/data_sync/membership/patch.ts is excluded by none and included by none
src/core/endpoints/data_sync/membership/remove.ts is excluded by none and included by none
src/core/endpoints/data_sync/membership/update.ts is excluded by none and included by none
src/core/endpoints/data_sync/relationship/create.ts is excluded by none and included by none
src/core/endpoints/data_sync/relationship/get-all.ts is excluded by none and included by none
src/core/endpoints/data_sync/relationship/get.ts is excluded by none and included by none
src/core/endpoints/data_sync/relationship/patch.ts is excluded by none and included by none
src/core/endpoints/data_sync/relationship/remove.ts is excluded by none and included by none
src/core/endpoints/data_sync/relationship/update.ts is excluded by none and included by none
src/core/endpoints/data_sync/user/create.ts is excluded by none and included by none
src/core/endpoints/data_sync/user/get-all.ts is excluded by none and included by none
src/core/endpoints/data_sync/user/get.ts is excluded by none and included by none
src/core/endpoints/data_sync/user/patch.ts is excluded by none and included by none
src/core/endpoints/data_sync/user/remove.ts is excluded by none and included by none
src/core/endpoints/data_sync/user/update.ts is excluded by none and included by none
src/core/pubnub-common.ts is excluded by none and included by none
src/core/pubnub-data-sync.ts is excluded by none and included by none
src/core/types/api/data-sync.ts is excluded by none and included by none
src/core/types/transport-request.ts is excluded by none and included by none
src/errors/pubnub-api-error.ts is excluded by none and included by none
src/transport/middleware.ts is excluded by none and included by none

📒 Files selected for processing (1)

.github/CODEOWNERS

…put accepted

Mohit Tejani added 9 commits May 22, 2026 13:27

pre-requisites for data_sync: handle error array during exceptional c…

0c83fb4

…ases, include PUT type where needed. update body include cases when body present while making http requests.

data-sync: channel endpoint apis

5502af9

data-sync:user endpoint apis

ea6da83

data-sync:membership endpoint apis

f128cbc

data-sync: entity endpoint apis

fbe3e3c

data-sync: entity-relationship endpoint apis

8734092

data-sync: apis domain object export, added operation constants, type…

1843165

… definition, updated pubnub-common file to have datasync apis support

updated dist/lib and codeowner files.

7a5a27f

fix lint

e82c07a

coderabbitai Bot reviewed May 22, 2026

View reviewed changes

Comment thread .github/CODEOWNERS Outdated

mohitpubnub and others added 6 commits May 22, 2026 13:40

Fix CODEOWNERS syntax for jguz-pubnub

f57123e

dataSync relationship api for getAll should have class and version in…

3876080

…put accepted

sync package.json with latest

adbfa29

fix type definition for getAllRelationships

fd1b9e4

Merge branch 'master' into CLEN-3438

66391b4

Merge branch 'master' into CLEN-3438

97152c2

mohitpubnub marked this pull request as ready for review June 25, 2026 08:33

mohitpubnub requested review from jguz-pubnub and parfeon as code owners June 25, 2026 08:33

Sunbelt Computer Software

PL/B Language Development and Support

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

data sync#498

data sync#498
mohitpubnub wants to merge 15 commits into
masterfrom
CLEN-3438

mohitpubnub commented May 22, 2026

Uh oh!

coderabbitai Bot commented May 22, 2026 •

edited

Loading

Review skipped

Walkthrough

Changes

Estimated code review effort

❌ Failed checks (2 inconclusive)

Uh oh!

pubnub-ops-terraform commented May 22, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Sunbelt Computer Software

PL/B Language Development and Support

Uh oh!

Conversation

mohitpubnub commented May 22, 2026

Uh oh!

coderabbitai Bot commented May 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Walkthrough

Changes

Estimated code review effort

❌ Failed checks (2 inconclusive)

Uh oh!

pubnub-ops-terraform commented May 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✅ Snyk checks have passed. No issues have been found so far.

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

coderabbitai Bot commented May 22, 2026 •

edited

Loading

pubnub-ops-terraform commented May 22, 2026 •

edited

Loading