@linxin666/dsh-pet 0.3.0 → 0.3.2

package/lib/types/client/PetSprite.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"PetSprite.d.ts","sourceRoot":"","sources":["../../../src/client/PetSprite.tsx"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAGH,OAAO,KAAK,EAAkE,SAAS,EAAE,WAAW,EAAE,MAAM,OAAO,CAAA;AAGnH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,kCAAkC,CAAA;AACnE,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,eAAe,CAAA;AACrD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,eAAe,CAAA;AACjD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAEnD,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAA;AAIjD,OAAO,EAAE,EAAE,EAAE,MAAM,cAAc,CAAA;AAGjC,wEAAwE;AACxE,MAAM,WAAW,cAAc;IAC7B,gDAAgD;IAChD,QAAQ,EAAE,YAAY,GAAG,IAAI,CAAA;IAC7B,8EAA8E;IAC9E,UAAU,EAAE,aAAa,CAAA;IACzB,qDAAqD;IACrD,OAAO,EAAE,gBAAgB,CAAA;IACzB,sCAAsC;IACtC,QAAQ,EAAE,WAAW,GAAG,IAAI,CAAA;IAC5B,8BAA8B;IAC9B,KAAK,EAAE,MAAM,IAAI,CAAA;IACjB,sCAAsC;IACtC,MAAM,EAAE,MAAM,IAAI,CAAA;IAClB,sCAAsC;IACtC,MAAM,EAAE,MAAM,IAAI,CAAA;IAClB,+BAA+B;IAC/B,SAAS,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,KAAK,IAAI,CAAA;IAClD,uDAAuD;IACvD,QAAQ,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAA;IAChC,4DAA4D;IAC5D,aAAa,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAA;IAC1C,2DAA2D;IAC3D,cAAc,EAAE,MAAM,IAAI,CAAA;IAC1B;;;;OAIG;IACH,MAAM,CAAC,EAAE,SAAS,CAAA;IAClB,+CAA+C;IAC/C,CAAC,EAAE,WAAW,CAAC,OAAO,EAAE,CAAC,CAAA;CAC1B;AAgGD;;;;;GAKG;AACH,wBAAgB,SAAS,CAAC,KAAK,EAAE,cAAc,GAAG,WAAW,CAiiB5D"}
+{"version":3,"file":"PetSprite.d.ts","sourceRoot":"","sources":["../../../src/client/PetSprite.tsx"],"names":[],"mappings":"AAAA;;;;;;;;;GASG;AAGH,OAAO,KAAK,EAAkE,SAAS,EAAE,WAAW,EAAE,MAAM,OAAO,CAAA;AAGnH,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,kCAAkC,CAAA;AACnE,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,eAAe,CAAA;AACrD,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,eAAe,CAAA;AACjD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAEnD,OAAO,KAAK,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAA;AAIjD,OAAO,EAAE,EAAE,EAAE,MAAM,cAAc,CAAA;AAGjC,wEAAwE;AACxE,MAAM,WAAW,cAAc;IAC7B,gDAAgD;IAChD,QAAQ,EAAE,YAAY,GAAG,IAAI,CAAA;IAC7B,8EAA8E;IAC9E,UAAU,EAAE,aAAa,CAAA;IACzB,qDAAqD;IACrD,OAAO,EAAE,gBAAgB,CAAA;IACzB,sCAAsC;IACtC,QAAQ,EAAE,WAAW,GAAG,IAAI,CAAA;IAC5B,8BAA8B;IAC9B,KAAK,EAAE,MAAM,IAAI,CAAA;IACjB,sCAAsC;IACtC,MAAM,EAAE,MAAM,IAAI,CAAA;IAClB,sCAAsC;IACtC,MAAM,EAAE,MAAM,IAAI,CAAA;IAClB,+BAA+B;IAC/B,SAAS,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,KAAK,IAAI,CAAA;IAClD,uDAAuD;IACvD,QAAQ,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAA;IAChC,4DAA4D;IAC5D,aAAa,EAAE,CAAC,SAAS,EAAE,MAAM,KAAK,IAAI,CAAA;IAC1C,2DAA2D;IAC3D,cAAc,EAAE,MAAM,IAAI,CAAA;IAC1B;;;;OAIG;IACH,MAAM,CAAC,EAAE,SAAS,CAAA;IAClB,+CAA+C;IAC/C,CAAC,EAAE,WAAW,CAAC,OAAO,EAAE,CAAC,CAAA;CAC1B;AAgGD;;;;;GAKG;AACH,wBAAgB,SAAS,CAAC,KAAK,EAAE,cAAc,GAAG,WAAW,CA0iB5D"}

package/lib/types/client/PetSprite.js

@@ -219,8 +219,9 @@ export function PetSprite(props) {
         // blank while the loop heat-up runs.
         const leadCol = track.frames[0];
         const lead = framePosition(cell, row, leadCol, scaleRef.current);
+        let lastPosStr = lead.x + 'px ' + lead.y + 'px';
         if (spriteRef.current !== null) {
-            spriteRef.current.style.backgroundPosition = lead.x + 'px ' + lead.y + 'px';
+            spriteRef.current.style.backgroundPosition = lastPosStr;
         }
         if (reduceMotion)
             return;
@@ -237,8 +238,12 @@ export function PetSprite(props) {
                 const currentTrack = trimTrack(tracks[current.animation], rows[currentRow] ?? tracks[current.animation].frames.length);
                 const col = currentTrack.frames[current.frameIndex];
                 const pos = framePosition(cell, currentRow, col, scaleRef.current);
-                if (spriteRef.current !== null) {
-                    spriteRef.current.style.backgroundPosition = pos.x + 'px ' + pos.y + 'px';
+                const posStr = pos.x + 'px ' + pos.y + 'px';
+                if (posStr !== lastPosStr) {
+                    lastPosStr = posStr;
+                    if (spriteRef.current !== null) {
+                        spriteRef.current.style.backgroundPosition = posStr;
+                    }
                 }
                 raf = requestAnimationFrame(tick);
                 return;
@@ -269,8 +274,12 @@ export function PetSprite(props) {
             }
             const col = track.frames[st.index];
             const pos = framePosition(cell, row, col, scaleRef.current);
-            if (spriteRef.current !== null) {
-                spriteRef.current.style.backgroundPosition = pos.x + 'px ' + pos.y + 'px';
+            const posStr = pos.x + 'px ' + pos.y + 'px';
+            if (posStr !== lastPosStr) {
+                lastPosStr = posStr;
+                if (spriteRef.current !== null) {
+                    spriteRef.current.style.backgroundPosition = posStr;
+                }
             }
             raf = requestAnimationFrame(tick);
         };

package/lib/types/client/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/client/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAwB,aAAa,EAAE,iBAAiB,EAAE,MAAM,wCAAwC,CAAA;AAgEnI,wEAAwE;AACxE,eAAO,MAAM,MAAM,UAA2E,CAAA;AAE9F,qEAAqE;AACrE,YAAY,EAAE,WAAW,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAA;AACxE,YAAY,EAAE,cAAc,EAAE,MAAM,iBAAiB,CAAA;AACrD,YAAY,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAA;AAC7D,YAAY,EAAE,mBAAmB,EAAE,oBAAoB,EAAE,MAAM,uBAAuB,CAAA;AACtF,YAAY,EAAE,uBAAuB,EAAE,MAAM,uBAAuB,CAAA;AACpE,YAAY,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAEnD,OAAO,QAAQ,qBAAqB,CAAC;IACnC,UAAU,OAAO;QACf;;;;WAIG;QACH,aAAa,CAAC,EAAE;YAAE,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,iBAAiB,CAAC,CAAC,CAAC,GAAG,aAAa,CAAC,CAAC,CAAC,CAAA;SAAE,CAAA;KAC1E;CACF;AAED;;;;;GAKG;AACH,wBAAgB,KAAK,CAAC,GAAG,EAAE,aAAa,GAAG,IAAI,CA0Q9C"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../../src/client/index.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAwB,aAAa,EAAE,iBAAiB,EAAE,MAAM,wCAAwC,CAAA;AAgEnI,wEAAwE;AACxE,eAAO,MAAM,MAAM,UAA2E,CAAA;AAE9F,qEAAqE;AACrE,YAAY,EAAE,WAAW,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAA;AACxE,YAAY,EAAE,cAAc,EAAE,MAAM,iBAAiB,CAAA;AACrD,YAAY,EAAE,UAAU,EAAE,WAAW,EAAE,MAAM,gBAAgB,CAAA;AAC7D,YAAY,EAAE,mBAAmB,EAAE,oBAAoB,EAAE,MAAM,uBAAuB,CAAA;AACtF,YAAY,EAAE,uBAAuB,EAAE,MAAM,uBAAuB,CAAA;AACpE,YAAY,EAAE,aAAa,EAAE,MAAM,gBAAgB,CAAA;AAEnD,OAAO,QAAQ,qBAAqB,CAAC;IACnC,UAAU,OAAO;QACf;;;;WAIG;QACH,aAAa,CAAC,EAAE;YAAE,IAAI,CAAC,CAAC,EAAE,IAAI,EAAE,iBAAiB,CAAC,CAAC,CAAC,GAAG,aAAa,CAAC,CAAC,CAAC,CAAA;SAAE,CAAA;KAC1E;CACF;AAED;;;;;GAKG;AACH,wBAAgB,KAAK,CAAC,GAAG,EAAE,aAAa,GAAG,IAAI,CAoR9C"}

package/lib/types/client/index.js

@@ -56,7 +56,14 @@ export const inject = ['slots', 'locale', 'connection', 'settingsScope', 'remote
  * @param ctx - client root context.
  */
 export function apply(ctx) {
-    ctx.effect(() => ctx.locale.register(NS, { zh, en }), 'pet: dictionaries');
+    ctx.effect(() => {
+        try {
+            return ctx.locale.register(NS, { zh, en });
+        }
+        catch {
+            return () => { };
+        }
+    }, 'pet: dictionaries');
     // Built-in renderers dispatch through the plugin-wide registry (pet-center
     // M3). Registration is idempotent (id wins), so re-applies stay clean.
     defaultPetRendererRegistry.register(live2dRenderer);
@@ -78,18 +85,23 @@ export function apply(ctx) {
     // The section entry owns the controller: unregistering it (fiber disposal,
     // hot reload) releases the scope subscription through petSettings.dispose.
     ctx.slots.inject('settings.section', () => {
-        const unregister = ctx.slots.register({
-            name: 'settings.section',
-            id: 'pet',
-            order: 130,
-            label: () => ctx.locale.bind('pet')('settings.title'),
-            locale: 'pet',
-            inject: () => petSettings.inject(),
-        }, PetSettingsSection);
-        return () => {
-            unregister();
-            petSettings.dispose();
-        };
+        try {
+            const unregister = ctx.slots.register({
+                name: 'settings.section',
+                id: 'pet',
+                order: 130,
+                label: () => ctx.locale.bind('pet')('settings.title'),
+                locale: 'pet',
+                inject: () => petSettings.inject(),
+            }, PetSettingsSection);
+            return () => {
+                unregister();
+                petSettings.dispose();
+            };
+        }
+        catch {
+            return () => { };
+        }
     });
     // The global pet entry, its store, and the poll loop live while the plugin
     // is enabled; toggling the setting off hides the pet and stops polling.

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@linxin666/dsh-pet",
   "description": "Multi-pet companion plugin for the dsh web GUI: a registry-driven floating pet that reacts to model activity, with per-pet naming, petting/feeding interactions and an affinity score",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "type": "module",
   "engines": {
     "node": "^22.19.0 || >=24.0.0"

package/src/client/PetSprite.test.tsx

@@ -576,6 +576,37 @@ describe('PetSprite definition-driven render', () => {
     act(() => { nextFrame?.(1_500) })
     expect(sprite.style.backgroundPosition).toBe('0px -960px')
   })
+
+  it('skips redundant backgroundPosition style assignments when frame coordinates do not change (issue #1013)', () => {
+    vi.spyOn(window, 'matchMedia').mockReturnValue({
+      matches: false,
+      media: '(prefers-reduced-motion: reduce)',
+      onchange: null,
+      addEventListener: () => {},
+      removeEventListener: () => {},
+      addListener: () => {},
+      removeListener: () => {},
+      dispatchEvent: () => false,
+    })
+    vi.spyOn(performance, 'now').mockReturnValue(0)
+    let nextFrame: FrameRequestCallback | undefined
+    vi.spyOn(window, 'requestAnimationFrame').mockImplementation(callback => {
+      nextFrame = callback
+      return 1
+    })
+    vi.spyOn(window, 'cancelAnimationFrame').mockImplementation(() => {})
+    renderPet({
+      snapshot: { ...snapshot, animation: 'idle', phase: 'idle' },
+    })
+    const sprite = screen.getByRole('button', { name: '鲸鱼娘' })
+    const styleSetterSpy = vi.spyOn(sprite.style, 'backgroundPosition', 'set')
+    // Idle frame 0 duration is 400ms. An intermediate tick at 16ms does not advance the frame.
+    act(() => { nextFrame?.(16) })
+    expect(styleSetterSpy).not.toHaveBeenCalled()
+    // A tick past 400ms advances to frame 1 and should set backgroundPosition once.
+    act(() => { nextFrame?.(410) })
+    expect(styleSetterSpy).toHaveBeenCalledTimes(1)
+  })
 })
 
 describe('PetSprite panel chrome from the voice pack (pet-center M4)', () => {

package/src/client/PetSprite.tsx

@@ -268,8 +268,9 @@ export function PetSprite(props: PetSpriteProps): ReactPortal {
     // blank while the loop heat-up runs.
     const leadCol = track.frames[0]!
     const lead = framePosition(cell, row, leadCol, scaleRef.current)
+    let lastPosStr = lead.x + 'px ' + lead.y + 'px'
     if (spriteRef.current !== null) {
-      spriteRef.current.style.backgroundPosition = lead.x + 'px ' + lead.y + 'px'
+      spriteRef.current.style.backgroundPosition = lastPosStr
     }
     if (reduceMotion) return
     let raf = 0
@@ -288,8 +289,12 @@ export function PetSprite(props: PetSpriteProps): ReactPortal {
         )
         const col = currentTrack.frames[current.frameIndex]!
         const pos = framePosition(cell, currentRow, col, scaleRef.current)
-        if (spriteRef.current !== null) {
-          spriteRef.current.style.backgroundPosition = pos.x + 'px ' + pos.y + 'px'
+        const posStr = pos.x + 'px ' + pos.y + 'px'
+        if (posStr !== lastPosStr) {
+          lastPosStr = posStr
+          if (spriteRef.current !== null) {
+            spriteRef.current.style.backgroundPosition = posStr
+          }
         }
         raf = requestAnimationFrame(tick)
         return
@@ -319,8 +324,12 @@ export function PetSprite(props: PetSpriteProps): ReactPortal {
       }
       const col = track.frames[st.index]!
       const pos = framePosition(cell, row, col, scaleRef.current)
-      if (spriteRef.current !== null) {
-        spriteRef.current.style.backgroundPosition = pos.x + 'px ' + pos.y + 'px'
+      const posStr = pos.x + 'px ' + pos.y + 'px'
+      if (posStr !== lastPosStr) {
+        lastPosStr = posStr
+        if (spriteRef.current !== null) {
+          spriteRef.current.style.backgroundPosition = posStr
+        }
       }
       raf = requestAnimationFrame(tick)
     }

package/src/client/index.ts

@@ -104,7 +104,13 @@ declare module '@deepseek-ai/cordis' {
  * @param ctx - client root context.
  */
 export function apply(ctx: ClientContext): void {
-  ctx.effect(() => ctx.locale.register(NS, { zh, en }), 'pet: dictionaries')
+  ctx.effect(() => {
+    try {
+      return ctx.locale.register(NS, { zh, en })
+    } catch {
+      return () => {}
+    }
+  }, 'pet: dictionaries')
 
   // Built-in renderers dispatch through the plugin-wide registry (pet-center
   // M3). Registration is idempotent (id wins), so re-applies stay clean.
@@ -129,17 +135,21 @@ export function apply(ctx: ClientContext): void {
   // The section entry owns the controller: unregistering it (fiber disposal,
   // hot reload) releases the scope subscription through petSettings.dispose.
   ctx.slots.inject('settings.section', () => {
-    const unregister = ctx.slots.register({
-      name: 'settings.section',
-      id: 'pet',
-      order: 130,
-      label: () => ctx.locale.bind('pet')('settings.title'),
-      locale: 'pet',
-      inject: () => petSettings.inject(),
-    }, PetSettingsSection)
-    return () => {
-      unregister()
-      petSettings.dispose()
+    try {
+      const unregister = ctx.slots.register({
+        name: 'settings.section',
+        id: 'pet',
+        order: 130,
+        label: () => ctx.locale.bind('pet')('settings.title'),
+        locale: 'pet',
+        inject: () => petSettings.inject(),
+      }, PetSettingsSection)
+      return () => {
+        unregister()
+        petSettings.dispose()
+      }
+    } catch {
+      return () => {}
     }
   })
 

package/src/client/pet-css.test.ts

@@ -51,4 +51,13 @@ describe('pet hover panel css', () => {
     expect(bridge).toContain('top: 100%')
     expect(bridge).toContain('bottom: auto')
   })
+
+  it('isolates the pet float container and sprite on their own compositor layer (issue #1013)', () => {
+    const float = css.match(/\.float\s*\{([^}]*)\}/)?.[1] ?? ''
+    expect(float).toContain('will-change: transform')
+    expect(float).toContain('contain: layout style')
+
+    const sprite = css.match(/\.sprite\s*\{([^}]*)\}/)?.[1] ?? ''
+    expect(sprite).toContain('contain: paint')
+  })
 })

package/src/client/pet.module.css

@@ -6,11 +6,14 @@
   display: flex;
   flex-direction: column;
   align-items: center;
+  will-change: transform;
+  contain: layout style;
 }
 
 .sprite {
   image-rendering: auto;
   touch-action: none;
+  contain: paint;
 }
 
 .spriteWrap {