@formicoidea/labre-core 0.33.0 → 0.34.2

package/dist/_pkgs/affine-components/open-doc-dropdown-menu/dropdown-menu.d.ts

@@ -20,9 +20,11 @@ export declare class OpenDocDropdownMenu extends OpenDocDropdownMenu_base {
         priority?: number;
     } & {
         label?: string;
+        labelWording?: import("../../affine-shared/services/index.js").ChromeWording;
         showLabel?: boolean;
         icon?: import("lit-html").TemplateResult;
         tooltip?: string | import("lit-html").TemplateResult;
+        tooltipWording?: import("../../affine-shared/services/index.js").ChromeWording;
         variant?: "destructive";
         disabled?: ((cx: ToolbarContext) => boolean) | boolean;
         content?: ((cx: ToolbarContext) => import("lit-html").TemplateResult | null) | (import("lit-html").TemplateResult | null);

package/dist/_pkgs/affine-components/resource/resource.js

@@ -63,6 +63,8 @@ export class ResourceController {
     updateState(state) {
         this.state$.value = { ...this.state$.value, ...state };
     }
+    // The explicit return type keeps declaration emit off signals-core's
+    // `DisposeFn`, whose `[Symbol.dispose]` member tsc cannot serialize.
     subscribe() {
         return effect(() => {
             const blobId = this.blobId$.value;

package/dist/_pkgs/affine-components/view-dropdown-menu/dropdown-menu.js

@@ -32,7 +32,7 @@ var __runInitializers = (this && this.__runInitializers) || function (thisArg, i
     }
     return useValue ? value : void 0;
 };
-import { ToolbarContext, } from '../../affine-shared/services/index.js';
+import { TOOLBAR_SWITCH_VIEW, toolbarActionLabel, ToolbarContext, translateKey, } from '../../affine-shared/services/index.js';
 import { SignalWatcher } from '../../global/lit/index.js';
 import { PropTypes, requiredProperties } from '../../std/index.js';
 import { LitElement } from 'lit';
@@ -86,13 +86,20 @@ let ViewDropdownMenu = (() => {
         set viewType$(value) { this.#viewType$_accessor_storage = value; }
         render() {
             const { actions, context, viewType$: { value: viewType }, } = this;
+            // These entries never went through `combine` — the group that owns them
+            // brings its own template and hands them over raw — so the i18n seam is
+            // applied here, with the same key the row would have used. `viewType` is
+            // resolved by the caller through the very same wordings, which is what
+            // keeps the "currently selected" comparison below a comparison of words.
+            const wordOf = (action) => toolbarActionLabel(context.std, action);
+            const switchView = translateKey(context.std, ...TOOLBAR_SWITCH_VIEW);
             return html `
       <editor-menu-button
         .contentPadding="${'8px'}"
         .button=${html `
           <editor-icon-button
-            aria-label="Switch view"
-            .tooltip="${'Switch view'}"
+            aria-label=${switchView}
+            .tooltip="${switchView}"
             .justify="${'space-between'}"
             .labelHeight="${'20px'}"
             .iconContainerWidth="${'110px'}"
@@ -107,17 +114,23 @@ let ViewDropdownMenu = (() => {
                 if (typeof action.when === 'function')
                     return action.when(context);
                 return action.when ?? true;
-            }), action => action.id, ({ id, label, disabled, run }) => html `
-              <editor-menu-action
-                aria-label="${ifDefined(label)}"
-                data-testid="${`link-to-${id}`}"
-                ?data-selected="${label === viewType}"
-                ?disabled="${ifDefined(typeof disabled === 'function' ? disabled(context) : disabled)}"
-                @click=${() => run?.(context)}
-              >
-                ${label}
-              </editor-menu-action>
-            `)}
+            }), action => action.id, action => {
+                const { id, disabled, run } = action;
+                const label = wordOf(action);
+                return html `
+                <editor-menu-action
+                  aria-label="${ifDefined(label)}"
+                  data-testid="${`link-to-${id}`}"
+                  ?data-selected="${label === viewType}"
+                  ?disabled="${ifDefined(typeof disabled === 'function'
+                    ? disabled(context)
+                    : disabled)}"
+                  @click=${() => run?.(context)}
+                >
+                  ${label}
+                </editor-menu-action>
+              `;
+            })}
         </div>
       </editor-menu-button>
     `;

package/dist/_pkgs/affine-inline-link/link-node/configs/toolbar.d.ts

@@ -20,19 +20,19 @@ export declare const builtinInlineLinkToolbarConfig: {
         id: string;
         actions: ({
             id: string;
-            label: string;
+            labelWording: import("../../../affine-shared/services/index.js").ChromeWording;
             disabled: true;
             run?: undefined;
             when?: undefined;
         } | {
             id: string;
-            label: string;
+            labelWording: import("../../../affine-shared/services/index.js").ChromeWording;
             run(ctx: import("../../../affine-shared/services/index.js").ToolbarContext): void;
             disabled?: undefined;
             when?: undefined;
         } | {
             id: string;
-            label: string;
+            labelWording: import("../../../affine-shared/services/index.js").ChromeWording;
             when(ctx: import("../../../affine-shared/services/index.js").ToolbarContext): boolean;
             run(ctx: import("../../../affine-shared/services/index.js").ToolbarContext): void;
             disabled?: undefined;
@@ -48,7 +48,7 @@ export declare const builtinInlineLinkToolbarConfig: {
     }, {
         readonly placement: ActionPlacement.More;
         readonly id: "c.delete";
-        readonly label: "Delete";
+        readonly labelWording: import("../../../affine-shared/services/index.js").ChromeWording;
         readonly icon: import("lit-html").TemplateResult<1>;
         readonly variant: "destructive";
         readonly run: (ctx: import("../../../affine-shared/services/index.js").ToolbarContext) => void;

package/dist/_pkgs/affine-inline-link/link-node/configs/toolbar.js

@@ -1,5 +1,5 @@
 import { toast } from '../../../affine-components/toast/index.js';
-import { ActionPlacement, EmbedIframeService, EmbedOptionProvider, } from '../../../affine-shared/services/index.js';
+import { ActionPlacement, EmbedIframeService, EmbedOptionProvider, TOOLBAR_CARD_VIEW, TOOLBAR_DELETE, TOOLBAR_EMBED_VIEW, TOOLBAR_INLINE_VIEW, toolbarActionLabel, } from '../../../affine-shared/services/index.js';
 import { CopyIcon, DeleteIcon, EditIcon, UnlinkIcon, } from '@blocksuite/icons/lit';
 import { BlockSelection } from '../../../std/index.js';
 import { signal } from '@preact/signals-core';
@@ -79,12 +79,12 @@ export const builtinInlineLinkToolbarConfig = {
             actions: [
                 {
                     id: 'inline',
-                    label: 'Inline view',
+                    labelWording: TOOLBAR_INLINE_VIEW,
                     disabled: true,
                 },
                 {
                     id: 'card',
-                    label: 'Card view',
+                    labelWording: TOOLBAR_CARD_VIEW,
                     run(ctx) {
                         const target = ctx.message$.peek()?.element;
                         if (!(target instanceof AffineLink))
@@ -133,7 +133,7 @@ export const builtinInlineLinkToolbarConfig = {
                 },
                 {
                     id: 'embed',
-                    label: 'Embed view',
+                    labelWording: TOOLBAR_EMBED_VIEW,
                     when(ctx) {
                         const target = ctx.message$.peek()?.element;
                         if (!(target instanceof AffineLink))
@@ -210,7 +210,7 @@ export const builtinInlineLinkToolbarConfig = {
                 if (!(target instanceof AffineLink))
                     return null;
                 const actions = this.actions.map(action => ({ ...action }));
-                const viewType$ = signal(actions[0].label);
+                const viewType$ = signal(toolbarActionLabel(ctx.std, actions[0]));
                 const onToggle = (e) => {
                     const opened = e.detail;
                     if (!opened)
@@ -280,7 +280,7 @@ export const builtinInlineLinkToolbarConfig = {
         {
             placement: ActionPlacement.More,
             id: 'c.delete',
-            label: 'Delete',
+            labelWording: TOOLBAR_DELETE,
             icon: DeleteIcon(),
             variant: 'destructive',
             run(ctx) {

package/dist/_pkgs/affine-inline-reference/reference-node/configs/toolbar.d.ts

@@ -7,17 +7,17 @@ export declare const builtinInlineReferenceToolbarConfig: {
         id: string;
         actions: ({
             id: string;
-            label: string;
+            labelWording: import("../../../affine-shared/services/index.js").ChromeWording;
             disabled: true;
             run?: undefined;
         } | {
             id: string;
-            label: string;
+            labelWording: import("../../../affine-shared/services/index.js").ChromeWording;
             run(ctx: import("../../../affine-shared/services/index.js").ToolbarContext): void;
             disabled?: undefined;
         } | {
             id: string;
-            label: string;
+            labelWording: import("../../../affine-shared/services/index.js").ChromeWording;
             disabled(ctx: import("../../../affine-shared/services/index.js").ToolbarContext): boolean;
             run(ctx: import("../../../affine-shared/services/index.js").ToolbarContext): void;
         })[];
@@ -26,7 +26,7 @@ export declare const builtinInlineReferenceToolbarConfig: {
     }, {
         readonly placement: ActionPlacement.More;
         readonly id: "c.delete";
-        readonly label: "Delete";
+        readonly labelWording: import("../../../affine-shared/services/index.js").ChromeWording;
         readonly icon: import("lit-html").TemplateResult<1>;
         readonly variant: "destructive";
         readonly run: (ctx: import("../../../affine-shared/services/index.js").ToolbarContext) => void;

package/dist/_pkgs/affine-inline-reference/reference-node/configs/toolbar.js

@@ -1,5 +1,5 @@
 import { notifyLinkedDocSwitchedToEmbed } from '../../../affine-components/notification/index.js';
-import { ActionPlacement, DocDisplayMetaProvider, } from '../../../affine-shared/services/index.js';
+import { ActionPlacement, DocDisplayMetaProvider, TOOLBAR_CARD_VIEW, TOOLBAR_DELETE, TOOLBAR_EMBED_VIEW, TOOLBAR_INLINE_VIEW, toolbarActionLabel, } from '../../../affine-shared/services/index.js';
 import { cloneReferenceInfoWithoutAliases, isInsideBlockByFlavour, } from '../../../affine-shared/utils/index.js';
 import { DeleteIcon } from '@blocksuite/icons/lit';
 import { BlockSelection } from '../../../std/index.js';
@@ -38,12 +38,12 @@ export const builtinInlineReferenceToolbarConfig = {
             actions: [
                 {
                     id: 'inline',
-                    label: 'Inline view',
+                    labelWording: TOOLBAR_INLINE_VIEW,
                     disabled: true,
                 },
                 {
                     id: 'card',
-                    label: 'Card view',
+                    labelWording: TOOLBAR_CARD_VIEW,
                     run(ctx) {
                         const target = ctx.message$.peek()?.element;
                         if (!(target instanceof AffineReference))
@@ -78,7 +78,7 @@ export const builtinInlineReferenceToolbarConfig = {
                 },
                 {
                     id: 'embed',
-                    label: 'Embed view',
+                    labelWording: TOOLBAR_EMBED_VIEW,
                     disabled(ctx) {
                         const target = ctx.message$.peek()?.element;
                         if (!(target instanceof AffineReference))
@@ -140,7 +140,7 @@ export const builtinInlineReferenceToolbarConfig = {
                 if (!(target instanceof AffineReference))
                     return null;
                 const actions = this.actions.map(action => ({ ...action }));
-                const viewType$ = signal(actions[0].label);
+                const viewType$ = signal(toolbarActionLabel(ctx.std, actions[0]));
                 const onToggle = (e) => {
                     const opened = e.detail;
                     if (!opened)
@@ -174,7 +174,7 @@ export const builtinInlineReferenceToolbarConfig = {
         {
             placement: ActionPlacement.More,
             id: 'c.delete',
-            label: 'Delete',
+            labelWording: TOOLBAR_DELETE,
             icon: DeleteIcon(),
             variant: 'destructive',
             run(ctx) {

package/dist/_pkgs/affine-model/elements/bpmn/pool.d.ts

@@ -1,6 +1,32 @@
 import type { SerializedXYWH } from '../../../global/gfx/index.js';
+import type { PointTestOptions } from '../../../std/gfx/index.js';
 import type { FrameworkBackgroundProps } from '../framework-background/index.js';
 import { FrameworkBackgroundElementModel } from '../framework-background/index.js';
+/**
+ * Width of the participant band — the left margin strip the pool name is
+ * written up, in model units.
+ *
+ * Declared HERE, in the model, and re-exported by `@labre/affine-gfx-bpmn`
+ * rather than the other way round. The pool's own hit test needs it (a pool is
+ * clickable by its title bands as well as its border, the bpmn.io convention),
+ * and `affine-model` cannot reach into `affine-block-surface` where the
+ * declaration is resolved. One number, owned by the layer that both the
+ * declaration and the hit test can read.
+ */
+export declare const POOL_BAND_WIDTH = 28;
+/**
+ * Width of a lane's own title band — the strip immediately inside the
+ * participant band, with the lane name turned on its side.
+ *
+ * Four units narrower than {@link POOL_BAND_WIDTH}, and for the same reason the
+ * lane font is two points smaller: the two strips sit side by side, so the
+ * subordinate one has to say so. Identical widths read as a single 56-unit
+ * gutter rather than as a participant containing lanes, which is exactly the
+ * relationship the picture has to carry.
+ *
+ * NO fill, divider only — what bpmn.io, Camunda and Visio all draw.
+ */
+export declare const POOL_LANE_BAND_WIDTH = 24;
 /**
  * One LANE (couloir) of a pool: a horizontal band the participant's own work is
  * divided into — "Sales", "Back office", "Automated".
@@ -84,4 +110,39 @@ export declare class BpmnPoolElementModel extends FrameworkBackgroundElementMode
      * primitive paints the dividers and the names from it.
      */
     accessor lanes: BpmnLane[] | undefined;
+    /**
+     * A pool is picked by its BORDER — and by its TITLE BANDS, which is the one
+     * carve-out the recette kept (issue #194).
+     *
+     * That is the bpmn.io convention, and it is the convention because the bands
+     * are the only part of a pool that is the pool rather than the process drawn
+     * on it: the participant strip on the left carries the participant's name,
+     * the lane strip beside it carries the lane names, and nothing is ever
+     * dropped on either. The flow area is where the work goes, so a click there
+     * belongs to whatever is under the pointer.
+     *
+     * ## Where the geometry comes from
+     *
+     * The two strips run the FULL height of the pool: the participant band is the
+     * whole left margin, and the lane bands partition that same height between
+     * them with no gap — so their union is one vertical strip
+     * `POOL_LANE_BAND_WIDTH` wide, whatever the lanes' individual weights are.
+     * Which means this test needs no lane arithmetic at all, only the two widths
+     * (declared above, and read from here by the framework declaration) and the
+     * answer to "does this pool have a usable lane".
+     *
+     * The zoom-grown rename targets — the touch-sized boxes a double-click aims
+     * at — are the VIEW's business and stay in `pool-hit.ts`; what is painted is
+     * what is selectable, which is the promise this test has to keep.
+     */
+    includesPoint(x: number, y: number, options?: PointTestOptions): boolean;
+    /**
+     * Whether at least one lane is drawn — the only thing the hit test needs to
+     * know about the partition.
+     *
+     * Mirrors what `backgroundInstanceZones` keeps (a string id, a finite size
+     * greater than zero): a malformed row is dropped there and paints no band, so
+     * it must not make a band clickable here either.
+     */
+    private get hasUsableLane();
 }

package/dist/_pkgs/affine-model/elements/bpmn/pool.js

@@ -32,8 +32,34 @@ var __runInitializers = (this && this.__runInitializers) || function (thisArg, i
     }
     return useValue ? value : void 0;
 };
+import { rotatePoint } from '../../../global/gfx/index.js';
 import { field } from '../../../std/gfx/index.js';
 import { FrameworkBackgroundElementModel } from '../framework-background/index.js';
+/**
+ * Width of the participant band — the left margin strip the pool name is
+ * written up, in model units.
+ *
+ * Declared HERE, in the model, and re-exported by `@labre/affine-gfx-bpmn`
+ * rather than the other way round. The pool's own hit test needs it (a pool is
+ * clickable by its title bands as well as its border, the bpmn.io convention),
+ * and `affine-model` cannot reach into `affine-block-surface` where the
+ * declaration is resolved. One number, owned by the layer that both the
+ * declaration and the hit test can read.
+ */
+export const POOL_BAND_WIDTH = 28;
+/**
+ * Width of a lane's own title band — the strip immediately inside the
+ * participant band, with the lane name turned on its side.
+ *
+ * Four units narrower than {@link POOL_BAND_WIDTH}, and for the same reason the
+ * lane font is two points smaller: the two strips sit side by side, so the
+ * subordinate one has to say so. Identical widths read as a single 56-unit
+ * gutter rather than as a participant containing lanes, which is exactly the
+ * relationship the picture has to carry.
+ *
+ * NO fill, divider only — what bpmn.io, Camunda and Visio all draw.
+ */
+export const POOL_LANE_BAND_WIDTH = 24;
 /**
  * A BPMN "pool": a participant container drawn as a rounded-rect frame with a
  * vertical name band on the left. It is a framework background like every other
@@ -131,6 +157,73 @@ let BpmnPoolElementModel = (() => {
          */
         get lanes() { return this.#lanes_accessor_storage; }
         set lanes(value) { this.#lanes_accessor_storage = value; }
+        /**
+         * A pool is picked by its BORDER — and by its TITLE BANDS, which is the one
+         * carve-out the recette kept (issue #194).
+         *
+         * That is the bpmn.io convention, and it is the convention because the bands
+         * are the only part of a pool that is the pool rather than the process drawn
+         * on it: the participant strip on the left carries the participant's name,
+         * the lane strip beside it carries the lane names, and nothing is ever
+         * dropped on either. The flow area is where the work goes, so a click there
+         * belongs to whatever is under the pointer.
+         *
+         * ## Where the geometry comes from
+         *
+         * The two strips run the FULL height of the pool: the participant band is the
+         * whole left margin, and the lane bands partition that same height between
+         * them with no gap — so their union is one vertical strip
+         * `POOL_LANE_BAND_WIDTH` wide, whatever the lanes' individual weights are.
+         * Which means this test needs no lane arithmetic at all, only the two widths
+         * (declared above, and read from here by the framework declaration) and the
+         * answer to "does this pool have a usable lane".
+         *
+         * The zoom-grown rename targets — the touch-sized boxes a double-click aims
+         * at — are the VIEW's business and stay in `pool-hit.ts`; what is painted is
+         * what is selectable, which is the promise this test has to keep.
+         */
+        includesPoint(x, y, options) {
+            if (super.includesPoint(x, y, options))
+                return true;
+            const [ex, ey, w, h] = this.deserializedXYWH;
+            // Element-local coordinates, undoing the element rotation about its centre
+            // — the bands are axis-aligned inside the pool, not on the canvas.
+            let lx = x - ex;
+            let ly = y - ey;
+            const rotate = this.rotate ?? 0;
+            if (rotate) {
+                const [ux, uy] = rotatePoint([x, y], [ex + w / 2, ey + h / 2], -rotate);
+                lx = ux - ex;
+                ly = uy - ey;
+            }
+            if (ly < 0 || ly > h || lx < 0)
+                return false;
+            // Clamped to a pool narrower than its own margin — the degenerate case the
+            // renderer clamps too.
+            const band = Math.min(POOL_BAND_WIDTH, w);
+            const lanes = this.hasUsableLane
+                ? Math.min(POOL_LANE_BAND_WIDTH, w - band)
+                : 0;
+            return lx <= band + Math.max(lanes, 0);
+        }
+        /**
+         * Whether at least one lane is drawn — the only thing the hit test needs to
+         * know about the partition.
+         *
+         * Mirrors what `backgroundInstanceZones` keeps (a string id, a finite size
+         * greater than zero): a malformed row is dropped there and paints no band, so
+         * it must not make a band clickable here either.
+         */
+        get hasUsableLane() {
+            const lanes = this.lanes;
+            if (!Array.isArray(lanes))
+                return false;
+            return lanes.some(lane => typeof lane?.id === 'string' &&
+                lane.id !== '' &&
+                typeof lane.size === 'number' &&
+                Number.isFinite(lane.size) &&
+                lane.size > 0);
+        }
         constructor() {
             super(...arguments);
             __runInitializers(this, _lanes_extraInitializers);

package/dist/_pkgs/affine-model/elements/cynefin/cynefin.d.ts

@@ -1,6 +1,6 @@
 import type { IVec, SerializedXYWH } from '../../../global/gfx/index.js';
 import { Bound } from '../../../global/gfx/index.js';
-import type { BaseElementProps } from '../../../std/gfx/index.js';
+import type { BaseElementProps, PointTestOptions } from '../../../std/gfx/index.js';
 import { GfxPrimitiveElementModel } from '../../../std/gfx/index.js';
 export type CynefinProps = BaseElementProps & {
     /** When false the resize handles are hidden — toggled from the toolbar. */
@@ -28,7 +28,12 @@ export declare class CynefinElementModel extends GfxPrimitiveElementModel<Cynefi
     containsBound(bounds: Bound): boolean;
     getLineIntersections(start: IVec, end: IVec): import("../../../global/gfx/index.js").PointLocation[] | null;
     getNearestPoint(point: IVec): IVec;
-    includesPoint(x: number, y: number): boolean;
+    /**
+     * Picked by its BORDER band — see {@link backgroundIncludesPoint}. The five
+     * domains are a picture the user drops elements onto, so the clicks inside
+     * the frame belong to those elements.
+     */
+    includesPoint(x: number, y: number, options?: PointTestOptions): boolean;
     accessor resizeEnabled: boolean;
     accessor showTitles: boolean;
     accessor showDescriptions: boolean;

package/dist/_pkgs/affine-model/elements/cynefin/cynefin.js

@@ -32,8 +32,9 @@ var __runInitializers = (this && this.__runInitializers) || function (thisArg, i
     }
     return useValue ? value : void 0;
 };
-import { Bound, getPointsFromBoundWithRotation, linePolygonIntersects, pointInPolygon, polygonNearestPoint, } from '../../../global/gfx/index.js';
+import { Bound, getPointsFromBoundWithRotation, linePolygonIntersects, polygonNearestPoint, } from '../../../global/gfx/index.js';
 import { field, GfxPrimitiveElementModel } from '../../../std/gfx/index.js';
+import { backgroundIncludesPoint } from '../framework-background/index.js';
 /**
  * A static "Liminal Cynefin" framework diagram: the hand-drawn five-domain
  * boundary (Complex / Complicated / Chaotic / Clear + central Confusion), the
@@ -98,9 +99,13 @@ let CynefinElementModel = (() => {
         getNearestPoint(point) {
             return polygonNearestPoint(Bound.deserialize(this.xywh).points, point);
         }
-        includesPoint(x, y) {
-            const points = getPointsFromBoundWithRotation(this);
-            return pointInPolygon([x, y], points);
+        /**
+         * Picked by its BORDER band — see {@link backgroundIncludesPoint}. The five
+         * domains are a picture the user drops elements onto, so the clicks inside
+         * the frame belong to those elements.
+         */
+        includesPoint(x, y, options) {
+            return backgroundIncludesPoint(this, x, y, options);
         }
         #resizeEnabled_accessor_storage = __runInitializers(this, _resizeEnabled_initializers, true);
         get resizeEnabled() { return this.#resizeEnabled_accessor_storage; }

package/dist/_pkgs/affine-model/elements/edgy/board.d.ts

@@ -1,7 +1,7 @@
 import type { IVec } from '../../../global/gfx/index.js';
 import { Bound } from '../../../global/gfx/index.js';
 import type { SerializedXYWH } from '../../../global/gfx/index.js';
-import type { BaseElementProps } from '../../../std/gfx/index.js';
+import type { BaseElementProps, PointTestOptions } from '../../../std/gfx/index.js';
 import { GfxPrimitiveElementModel } from '../../../std/gfx/index.js';
 export type EdgyBoardProps = BaseElementProps & {
     /** When false the resize handles are hidden — toggled from the toolbar. */
@@ -24,7 +24,12 @@ export declare class EdgyBoardElementModel extends GfxPrimitiveElementModel<Edgy
     containsBound(bounds: Bound): boolean;
     getLineIntersections(start: IVec, end: IVec): import("../../../global/gfx/index.js").PointLocation[] | null;
     getNearestPoint(point: IVec): IVec;
-    includesPoint(x: number, y: number): boolean;
+    /**
+     * Picked by its BORDER band: a board is a passive canvas and the clicks over
+     * its area belong to what the user laid on it. Same test every framework
+     * background runs — see {@link backgroundIncludesPoint}.
+     */
+    includesPoint(x: number, y: number, options?: PointTestOptions): boolean;
     accessor resizeEnabled: boolean;
     accessor spotlightEnabled: boolean;
     accessor rotate: number;

package/dist/_pkgs/affine-model/elements/edgy/board.js

@@ -32,8 +32,9 @@ var __runInitializers = (this && this.__runInitializers) || function (thisArg, i
     }
     return useValue ? value : void 0;
 };
-import { Bound, getPointsFromBoundWithRotation, linePolygonIntersects, pointInPolygon, polygonNearestPoint, } from '../../../global/gfx/index.js';
+import { Bound, getPointsFromBoundWithRotation, linePolygonIntersects, polygonNearestPoint, } from '../../../global/gfx/index.js';
 import { field, GfxPrimitiveElementModel } from '../../../std/gfx/index.js';
+import { backgroundIncludesPoint } from '../framework-background/index.js';
 /**
  * A blank white board for free-form EDGY modelling: the user places EDGY base
  * shapes and connectors on top of it and the board grants them the dependency
@@ -87,9 +88,13 @@ let EdgyBoardElementModel = (() => {
         getNearestPoint(point) {
             return polygonNearestPoint(Bound.deserialize(this.xywh).points, point);
         }
-        includesPoint(x, y) {
-            const points = getPointsFromBoundWithRotation(this);
-            return pointInPolygon([x, y], points);
+        /**
+         * Picked by its BORDER band: a board is a passive canvas and the clicks over
+         * its area belong to what the user laid on it. Same test every framework
+         * background runs — see {@link backgroundIncludesPoint}.
+         */
+        includesPoint(x, y, options) {
+            return backgroundIncludesPoint(this, x, y, options);
         }
         #resizeEnabled_accessor_storage = __runInitializers(this, _resizeEnabled_initializers, true);
         get resizeEnabled() { return this.#resizeEnabled_accessor_storage; }

package/dist/_pkgs/affine-model/elements/edgy/facets.d.ts

@@ -1,6 +1,6 @@
 import type { IVec, SerializedXYWH } from '../../../global/gfx/index.js';
 import { Bound } from '../../../global/gfx/index.js';
-import type { BaseElementProps } from '../../../std/gfx/index.js';
+import type { BaseElementProps, PointTestOptions } from '../../../std/gfx/index.js';
 import { GfxPrimitiveElementModel } from '../../../std/gfx/index.js';
 export type EdgyFacetsProps = BaseElementProps & {
     /** When false the resize handles are hidden — toggled from the toolbar. */
@@ -14,7 +14,15 @@ export type EdgyFacetsProps = BaseElementProps & {
      * margins only exist for the facet name labels — pointless when hidden).
      */
     cropToCircles?: boolean;
-    /** When false the diagram stops granting the hover spotlight. */
+    /**
+     * INERT since #195 — kept for backward compatibility only.
+     *
+     * The hover spotlight is board logic: it is granted by the EDGY board
+     * (`edgyBoard`), never by this drawing, so nothing reads this field on a
+     * facets Venn any more. It stays declared because documents written before
+     * the change carry it, and an undeclared property would not survive a load
+     * (see the schema red zone in `CLAUDE.md`). Do not read it, do not remove it.
+     */
     spotlightEnabled?: boolean;
     identityLabel?: string;
     architectureLabel?: string;
@@ -41,11 +49,20 @@ export declare class EdgyFacetsElementModel extends GfxPrimitiveElementModel<Edg
     containsBound(bounds: Bound): boolean;
     getLineIntersections(start: IVec, end: IVec): import("../../../global/gfx/index.js").PointLocation[] | null;
     getNearestPoint(point: IVec): IVec;
-    includesPoint(x: number, y: number): boolean;
+    /**
+     * Picked by its BORDER band — see {@link backgroundIncludesPoint}.
+     *
+     * The facet LABELS stay double-clickable all the same: their zones are added
+     * by `EdgyView.includesPoint`, which is what the pointer router consults, so
+     * the rename target lives with the code that draws it and the model layer
+     * stays free of it.
+     */
+    includesPoint(x: number, y: number, options?: PointTestOptions): boolean;
     accessor resizeEnabled: boolean;
     accessor showLabels: boolean;
     accessor showPictos: boolean;
     accessor cropToCircles: boolean;
+    /** Inert — see {@link EdgyFacetsProps.spotlightEnabled} (#195). */
     accessor spotlightEnabled: boolean;
     accessor identityLabel: string;
     accessor architectureLabel: string;

package/dist/_pkgs/affine-model/elements/edgy/facets.js

@@ -32,8 +32,9 @@ var __runInitializers = (this && this.__runInitializers) || function (thisArg, i
     }
     return useValue ? value : void 0;
 };
-import { Bound, getPointsFromBoundWithRotation, linePolygonIntersects, pointInPolygon, polygonNearestPoint, } from '../../../global/gfx/index.js';
+import { Bound, getPointsFromBoundWithRotation, linePolygonIntersects, polygonNearestPoint, } from '../../../global/gfx/index.js';
 import { field, GfxPrimitiveElementModel } from '../../../std/gfx/index.js';
+import { backgroundIncludesPoint } from '../framework-background/index.js';
 /**
  * A static "EDGY Enterprise Design Facets" diagram: three overlapping circles
  * (Identity / Architecture / Experience) with their three pairwise
@@ -123,9 +124,16 @@ let EdgyFacetsElementModel = (() => {
         getNearestPoint(point) {
             return polygonNearestPoint(Bound.deserialize(this.xywh).points, point);
         }
-        includesPoint(x, y) {
-            const points = getPointsFromBoundWithRotation(this);
-            return pointInPolygon([x, y], points);
+        /**
+         * Picked by its BORDER band — see {@link backgroundIncludesPoint}.
+         *
+         * The facet LABELS stay double-clickable all the same: their zones are added
+         * by `EdgyView.includesPoint`, which is what the pointer router consults, so
+         * the rename target lives with the code that draws it and the model layer
+         * stays free of it.
+         */
+        includesPoint(x, y, options) {
+            return backgroundIncludesPoint(this, x, y, options);
         }
         #resizeEnabled_accessor_storage = __runInitializers(this, _resizeEnabled_initializers, true);
         get resizeEnabled() { return this.#resizeEnabled_accessor_storage; }
@@ -140,6 +148,7 @@ let EdgyFacetsElementModel = (() => {
         get cropToCircles() { return this.#cropToCircles_accessor_storage; }
         set cropToCircles(value) { this.#cropToCircles_accessor_storage = value; }
         #spotlightEnabled_accessor_storage = (__runInitializers(this, _cropToCircles_extraInitializers), __runInitializers(this, _spotlightEnabled_initializers, true));
+        /** Inert — see {@link EdgyFacetsProps.spotlightEnabled} (#195). */
         get spotlightEnabled() { return this.#spotlightEnabled_accessor_storage; }
         set spotlightEnabled(value) { this.#spotlightEnabled_accessor_storage = value; }
         #identityLabel_accessor_storage = (__runInitializers(this, _spotlightEnabled_extraInitializers), __runInitializers(this, _identityLabel_initializers, 'Identity'));

package/dist/_pkgs/affine-model/elements/estuarine/estuarine.d.ts

@@ -1,6 +1,6 @@
 import type { IVec, SerializedXYWH } from '../../../global/gfx/index.js';
 import { Bound } from '../../../global/gfx/index.js';
-import type { BaseElementProps } from '../../../std/gfx/index.js';
+import type { BaseElementProps, PointTestOptions } from '../../../std/gfx/index.js';
 import { GfxPrimitiveElementModel } from '../../../std/gfx/index.js';
 export type EstuarineProps = BaseElementProps & {
     /** When false the resize handles are hidden — toggled from the toolbar. */
@@ -26,7 +26,12 @@ export declare class EstuarineElementModel extends GfxPrimitiveElementModel<Estu
     containsBound(bounds: Bound): boolean;
     getLineIntersections(start: IVec, end: IVec): import("../../../global/gfx/index.js").PointLocation[] | null;
     getNearestPoint(point: IVec): IVec;
-    includesPoint(x: number, y: number): boolean;
+    /**
+     * Picked by its BORDER band — see {@link backgroundIncludesPoint}. The map is
+     * a space constraint nodes are dropped into, so the clicks inside it are
+     * theirs.
+     */
+    includesPoint(x: number, y: number, options?: PointTestOptions): boolean;
     accessor resizeEnabled: boolean;
     accessor showLiminal: boolean;
     accessor showVolatile: boolean;