Skip to content

(protocol squad) main branch pr - #793

Open
leanmendoza wants to merge 113 commits into
mainfrom
protocol-squad
Open

(protocol squad) main branch pr#793
leanmendoza wants to merge 113 commits into
mainfrom
protocol-squad

Conversation

@leanmendoza

@leanmendoza leanmendoza commented Oct 17, 2023

Copy link
Copy Markdown
Contributor
  • Install the protocol-squad version in your scene by running npm i @dcl/sdk@protocol-squad in your scene root folder.
  • Run npm start to run Preview in Bevy

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Oct 17, 2023

Copy link
Copy Markdown

Deploying js-sdk-toolchain with  Cloudflare Pages  Cloudflare Pages

Latest commit: 0c215c3
Status: ✅  Deploy successful!
Preview URL: https://3fe37d48.js-sdk-toolchain.pages.dev
Branch Preview URL: https://protocol-squad.js-sdk-toolchain.pages.dev

View logs

@codecov

codecov Bot commented Oct 17, 2023

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 58.28402% with 141 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.06%. Comparing base (b281f0c) to head (8a3e6ef).
⚠️ Report is 44 commits behind head on main.

Files with missing lines Patch % Lines
...ckages/@dcl/sdk-commands/src/logic/dao-explorer.ts 19.27% 58 Missing and 9 partials ⚠️
packages/@dcl/ecs/src/systems/events.ts 79.59% 20 Missing ⚠️
...ages/@dcl/sdk-commands/src/commands/start/index.ts 5.00% 18 Missing and 1 partial ⚠️
packages/@dcl/react-ecs/src/reconciler/index.ts 51.51% 16 Missing ⚠️
...EntityInspector/MaterialInspector/Texture/utils.ts 14.28% 6 Missing ⚠️
packages/@dcl/react-ecs/src/system.ts 58.33% 5 Missing ⚠️
packages/@dcl/sdk/src/index.ts 0.00% 3 Missing and 1 partial ⚠️
.../ecs/src/components/extended/AvatarEquippedData.ts 91.66% 1 Missing ⚠️
...es/@dcl/ecs/src/components/extended/AvatarShape.ts 91.66% 1 Missing ⚠️
...dcl/react-ecs/src/components/uiBackground/utils.ts 50.00% 1 Missing ⚠️
... and 1 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #793      +/-   ##
==========================================
- Coverage   62.33%   62.06%   -0.28%     
==========================================
  Files         618      622       +4     
  Lines       20645    20876     +231     
  Branches     2518     2593      +75     
==========================================
+ Hits        12870    12956      +86     
- Misses       7338     7469     +131     
- Partials      437      451      +14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@leanmendoza leanmendoza changed the title (do not merge) protocol squad: experimental main branch (protocol squad) main branch Nov 29, 2023
@leanmendoza leanmendoza changed the title (protocol squad) main branch (protocol squad) main branch pr Nov 29, 2023
@decentraland decentraland deleted a comment from github-actions Bot May 7, 2024
* feat: add bevy in start command

* add locaiton

* only pass base url

* bevy versioning

* fix urls and rmdir

* do not open browser when dao-explorer is specified
* experimental cinematic camera

* skip test

* fix typo

* wip

* update protocol

* update bevy release

* update protocol
* bump protocol, update after build and add to react

* add customEntryPoint to sdk-commands start

* add wrap

* add to react

* update build

* update @dcl/protocol
@kuruk-mm
kuruk-mm marked this pull request as ready for review June 19, 2025 18:04
robtfm and others added 16 commits July 23, 2025 19:03
* multi-button ui

* fix callback type

* partial

* lint

* leave public Callback type unchanged, make new MultiCallback type

* lint + doc

* add event arg to single callback

* onInputXXX

* forgot about iaAny

* fix ternary precedence

* fix comparison

* use correct event ordering for getInputCommands()

* fix button cast

* fix doc comments

* lints

* build...

* rest of the lints

* fix tests

* fix lint

* fix initial empty pointerEvents

* update snapshot

---------

Co-authored-by: Lean Mendoza <8042536+leanmendoza@users.noreply.github.com>
* feat: dynamic lights (#1186)

* updated proto package and added tests for light source

* updated proto package

* 'make build' + 'make update-snapshots'

* corrected LightSource test based on Material tests

* removed test code

* final version of LightSource serialization test

* ran 'make lint-fix'

* added light source helper

* updated protocol package

* added missing helper tests for LightSource

* ran 'make lint-fix'

---------

Co-authored-by: Pravus <pravusjif@gmail.com>

* build with latest psquad package

* update snapshots

---------

Co-authored-by: daniele-dcl <daniele.marotta@decentraland.org>
Co-authored-by: Pravus <pravusjif@gmail.com>
* clear/get focus

* use @dcl/protocol with main branch updates

* fix

* fixes

* lint

---------

Co-authored-by: Mateo "Kuruk" Miccino <mateomiccino@gmail.com>
* fix textureRenderer wrong parameter must be the function component instead of the result object of it

* update snapshots

* remove test-inspector

---------

Co-authored-by: pabloes <estornut@gmail.com>
# Conflicts:
#	package-lock.json
#	package.json
#	packages/@dcl/ecs/src/components/types.ts
#	packages/@dcl/playground-assets/etc/playground-assets.api.md
#	packages/@dcl/react-ecs/src/system.ts
#	packages/@dcl/sdk-commands/package-lock.json
#	packages/@dcl/sdk-commands/package.json
#	packages/@dcl/sdk-commands/src/commands/start/index.ts
#	test/snapshots/development-bundles/static-scene.test.ts.crdt
#	test/snapshots/development-bundles/testing-fw.test.ts.crdt
#	test/snapshots/development-bundles/two-way-crdt.test.ts.crdt
#	test/snapshots/package-lock.json
#	test/snapshots/production-bundles/append-value-crdt.ts.crdt
#	test/snapshots/production-bundles/billboard.ts.crdt
#	test/snapshots/production-bundles/cube-deleted.ts.crdt
#	test/snapshots/production-bundles/cube.ts.crdt
#	test/snapshots/production-bundles/cubes.ts.crdt
#	test/snapshots/production-bundles/pointer-events.ts.crdt
#	test/snapshots/production-bundles/schema-components.ts.crdt
#	test/snapshots/production-bundles/ui.ts.crdt
#	test/snapshots/production-bundles/with-main-function.ts.crdt
robtfm and others added 12 commits February 23, 2026 14:04
* update protocol

* add ci on pr

* update raycast spec

* update ids

* update snapshots

* skip playground

* add raycast params to helpers

* add IA_MODIFIER
- Update @dcl/protocol to local build with WalkPlayerTo RPC and new walk fields
- Add walkPlayerTo mock to scene-executor RestrictedActions
- Regenerate snapshots for memory usage change from new types
Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…1421)

propsChanged diffed only the keys present in prevProps, so an optional
prop that appears for the first time (absent from prevProps, present in
nextProps) was silently dropped on its first change.

scrollPosition is the canonical case: while undefined it is omitted from
the parsed UiTransform, so the first `undefined -> reference` change
produced no CRDT update at all. This is why the first scroll-to-target
on a scrollable was lost while every subsequent change worked, and why a
non-null initial scrollPosition avoided it.

Add a second loop over nextProps for keys not in prevProps. This is
allocation-free (no Object.keys/Set) to keep this per-frame hot path
cheap.
npm publishing for the @dcl packages moved to trusted publishing (OIDC),
which needs npm >= 11.5.1. Node 20's npm 10 falls back to the revoked
NPM_TOKEN and every publish since May has failed with E404, so the
protocol-squad npm tag is stuck at commit 83012ab (2026-04-27). Matches
the Node pin main took in #1409/#1425.
* fix protocol-squad branch tests

* fix(tests): pin typescript in dcl-test-lib-integration fixture

`typescript: "*"` now resolves to TypeScript 6.0, which turns deprecated
compiler options (downlevelIteration, moduleResolution=node10) into hard
errors and also flags the missing rootDir (TS5101/TS5107/TS5011). That broke
`tsc` for the lib fixture, so its bin/ was never produced and the dependent
scene builds failed to resolve `dcl-test-lib-integration`, failing
compile-lib-and-scene.spec.ts in CI.

Mirror the fix already on `main`: pin typescript to ^5.0.2 and set rootDir.
protocol-squad had drifted 65 commits behind main, missing correctness
fixes (entity-id collision #1332, CRDT suppression #1359, UiInput echo
#1331) and features (ParticleSystem, physics forces, --no-client).

Conflict resolutions, by intent:
- @dcl/protocol stays THIS branch's tarball pin (root + sdk-commands)
- start/index.ts takes main's launch machinery (--no-client, --mcp,
  forwarded params, mobile QR back to opt-in) but keeps bevy-web
  preview as this branch's default; main's sortedURLs[0] adapted to
  the object shape this file uses
- events.ts is a semantic port, not a splice: main's proximity events
  (InteractionType threading, PET_PROXIMITY_* mappings, setters)
  re-implemented in this branch's per-button multi-callback idiom
- AudioAnalysis is EXCLUDED: it needs audio_analysis.proto, which this
  branch's protocol does not carry yet; re-add when the proto lands
- Component specs that main wrote against its newer protocol revert to
  this branch's versions (VirtualCamera fov etc.); ParticleSystem.spec
  adapted to this protocol's unwrapped repeated bursts
- CLAUDE.md -> @AGENTS.md pointer (main's convention); the Unity
  renderer workflow stays deleted on this branch
- Locks, playground api.md and .crdt snapshots regenerated, not merged

make build && make test: 153 suites, 1144 tests green.
…quad

Merge main into protocol-squad (keeping this branch's protocol)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants