scratch-blocks 2.1.12 → 2.1.14

package/dist/types/src/blocks/procedures.d.ts

@@ -30,6 +30,7 @@ type ConnectionMap = Record<string, {
 /**
  * Possible types for procedure arguments.
  */
+type ArgumentDefault = string | number | boolean | null;
 declare enum ArgumentType {
     STRING = "s",
     NUMBER = "n",
@@ -50,7 +51,7 @@ interface ProcedureBlock extends Blockly.BlockSvg {
 }
 export interface ProcedureDeclarationBlock extends ProcedureBlock {
     displayNames_: string[];
-    argumentDefaults_: string[];
+    argumentDefaults_: ArgumentDefault[];
     removeFieldCallback: (field: Blockly.Field) => void;
     createArgumentEditor_: (argumentType: ArgumentType, displayName: string) => Blockly.BlockSvg;
     focusLastEditor_: () => void;

package/dist/types/src/blocks/procedures.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"procedures.d.ts","sourceRoot":"","sources":["../../../../src/blocks/procedures.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AACH;;GAEG;AACH,OAAO,KAAK,OAAO,MAAM,cAAc,CAAA;AAIvC;;GAEG;AACH,KAAK,aAAa,GAAG,MAAM,CACzB,MAAM,EACN;IACE,MAAM,EAAE,OAAO,GAAG,SAAS,CAAA;IAC3B,KAAK,EAAE,OAAO,CAAC,QAAQ,GAAG,IAAI,CAAA;CAC/B,GAAG,IAAI,CACT,CAAA;AAED;;GAEG;AACH,aAAK,YAAY;IACf,MAAM,MAAM;IACZ,MAAM,MAAM;IACZ,OAAO,MAAM;CACd;AA8vCD,UAAU,cAAe,SAAQ,OAAO,CAAC,QAAQ;IAC/C,SAAS,EAAE,MAAM,CAAA;IACjB,YAAY,EAAE,MAAM,EAAE,CAAA;IACtB,KAAK,EAAE,OAAO,CAAA;IACd,WAAW,EAAE,MAAM,MAAM,CAAA;IACzB,gBAAgB,EAAE,MAAM,IAAI,CAAA;IAC5B,oBAAoB,EAAE,MAAM,aAAa,CAAA;IACzC,sBAAsB,EAAE,CAAC,aAAa,EAAE,aAAa,KAAK,IAAI,CAAA;IAC9D,gBAAgB,EAAE,CAAC,aAAa,EAAE,aAAa,KAAK,IAAI,CAAA;IACxD,cAAc,EAAE,MAAM,IAAI,CAAA;IAC1B,iBAAiB,EAAE,CACjB,IAAI,EAAE,YAAY,EAClB,KAAK,EAAE,MAAM,EACb,aAAa,EAAE,aAAa,EAC5B,EAAE,EAAE,MAAM,EACV,KAAK,EAAE,OAAO,CAAC,KAAK,KACjB,IAAI,CAAA;IACT,kBAAkB,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAA;CAC3C;AAED,MAAM,WAAW,yBAA0B,SAAQ,cAAc;IAC/D,aAAa,EAAE,MAAM,EAAE,CAAA;IACvB,iBAAiB,EAAE,MAAM,EAAE,CAAA;IAC3B,mBAAmB,EAAE,CAAC,KAAK,EAAE,OAAO,CAAC,KAAK,KAAK,IAAI,CAAA;IACnD,qBAAqB,EAAE,CAAC,YAAY,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,KAAK,OAAO,CAAC,QAAQ,CAAA;IAC5F,gBAAgB,EAAE,MAAM,IAAI,CAAA;IAC5B,OAAO,EAAE,MAAM,OAAO,CAAA;IACtB,OAAO,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAA;IAChC,gBAAgB,EAAE,MAAM,IAAI,CAAA;IAC5B,kBAAkB,EAAE,MAAM,IAAI,CAAA;IAC9B,uBAAuB,EAAE,MAAM,IAAI,CAAA;IACnC,UAAU,EAAE,MAAM,IAAI,CAAA;CACvB"}
+{"version":3,"file":"procedures.d.ts","sourceRoot":"","sources":["../../../../src/blocks/procedures.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;GAiBG;AACH;;GAEG;AACH,OAAO,KAAK,OAAO,MAAM,cAAc,CAAA;AAIvC;;GAEG;AACH,KAAK,aAAa,GAAG,MAAM,CACzB,MAAM,EACN;IACE,MAAM,EAAE,OAAO,GAAG,SAAS,CAAA;IAC3B,KAAK,EAAE,OAAO,CAAC,QAAQ,GAAG,IAAI,CAAA;CAC/B,GAAG,IAAI,CACT,CAAA;AAED;;GAEG;AACH,KAAK,eAAe,GAAG,MAAM,GAAG,MAAM,GAAG,OAAO,GAAG,IAAI,CAAA;AAEvD,aAAK,YAAY;IACf,MAAM,MAAM;IACZ,MAAM,MAAM;IACZ,OAAO,MAAM;CACd;AAgwCD,UAAU,cAAe,SAAQ,OAAO,CAAC,QAAQ;IAC/C,SAAS,EAAE,MAAM,CAAA;IACjB,YAAY,EAAE,MAAM,EAAE,CAAA;IACtB,KAAK,EAAE,OAAO,CAAA;IACd,WAAW,EAAE,MAAM,MAAM,CAAA;IACzB,gBAAgB,EAAE,MAAM,IAAI,CAAA;IAC5B,oBAAoB,EAAE,MAAM,aAAa,CAAA;IACzC,sBAAsB,EAAE,CAAC,aAAa,EAAE,aAAa,KAAK,IAAI,CAAA;IAC9D,gBAAgB,EAAE,CAAC,aAAa,EAAE,aAAa,KAAK,IAAI,CAAA;IACxD,cAAc,EAAE,MAAM,IAAI,CAAA;IAC1B,iBAAiB,EAAE,CACjB,IAAI,EAAE,YAAY,EAClB,KAAK,EAAE,MAAM,EACb,aAAa,EAAE,aAAa,EAC5B,EAAE,EAAE,MAAM,EACV,KAAK,EAAE,OAAO,CAAC,KAAK,KACjB,IAAI,CAAA;IACT,kBAAkB,EAAE,CAAC,IAAI,EAAE,MAAM,KAAK,IAAI,CAAA;CAC3C;AAED,MAAM,WAAW,yBAA0B,SAAQ,cAAc;IAC/D,aAAa,EAAE,MAAM,EAAE,CAAA;IACvB,iBAAiB,EAAE,eAAe,EAAE,CAAA;IACpC,mBAAmB,EAAE,CAAC,KAAK,EAAE,OAAO,CAAC,KAAK,KAAK,IAAI,CAAA;IACnD,qBAAqB,EAAE,CAAC,YAAY,EAAE,YAAY,EAAE,WAAW,EAAE,MAAM,KAAK,OAAO,CAAC,QAAQ,CAAA;IAC5F,gBAAgB,EAAE,MAAM,IAAI,CAAA;IAC5B,OAAO,EAAE,MAAM,OAAO,CAAA;IACtB,OAAO,EAAE,CAAC,IAAI,EAAE,OAAO,KAAK,IAAI,CAAA;IAChC,gBAAgB,EAAE,MAAM,IAAI,CAAA;IAC5B,kBAAkB,EAAE,MAAM,IAAI,CAAA;IAC9B,uBAAuB,EAAE,MAAM,IAAI,CAAA;IACnC,UAAU,EAAE,MAAM,IAAI,CAAA;CACvB"}

package/dist/types/src/fields/scratch_field_dropdown.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"scratch_field_dropdown.d.ts","sourceRoot":"","sources":["../../../../src/fields/scratch_field_dropdown.ts"],"names":[],"mappings":"AAqCA;;GAEG;AACH,wBAAgB,4BAA4B,SAG3C"}
+{"version":3,"file":"scratch_field_dropdown.d.ts","sourceRoot":"","sources":["../../../../src/fields/scratch_field_dropdown.ts"],"names":[],"mappings":"AAiFA;;GAEG;AACH,wBAAgB,4BAA4B,SAG3C"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "scratch-blocks",
-  "version": "2.1.12",
+  "version": "2.1.14",
   "description": "Scratch Blocks is a library for building creative computing interfaces.",
   "author": "Massachusetts Institute of Technology",
   "license": "Apache-2.0",
@@ -41,7 +41,7 @@
     "@vitest/browser": "4.1.4",
     "@vitest/browser-playwright": "4.1.4",
     "eslint": "9.39.4",
-    "eslint-config-scratch": "14.1.7",
+    "eslint-config-scratch": "14.1.8",
     "husky": "9.1.7",
     "playwright": "1.59.1",
     "prettier": "3.8.2",

package/src/blocks/procedures.ts

@@ -37,6 +37,8 @@ type ConnectionMap = Record<
 /**
  * Possible types for procedure arguments.
  */
+type ArgumentDefault = string | number | boolean | null
+
 enum ArgumentType {
   STRING = 's',
   NUMBER = 'n',
@@ -75,39 +77,17 @@ function getRequiredMutationAttribute(xmlElement: Element, name: string): string
   return value
 }
 
-/**
- * Parse a required mutation attribute as JSON, then validate its type.
- * @param xmlElement The mutation element that should contain required attributes.
- * @param name The specific mutation attribute to retrieve and parse.
- * @param parse Validates and narrows the parsed JSON value.
- * @returns Parsed and validated mutation attribute value.
- */
-function parseRequiredMutationJson<T>(
-  xmlElement: Element,
-  name: string,
-  parse: (value: unknown, name: string) => T,
-): T {
-  const rawValue = getRequiredMutationAttribute(xmlElement, name)
-  let parsedValue: unknown
-  try {
-    parsedValue = JSON.parse(rawValue)
-  } catch {
-    throw new Error(`Invalid JSON in mutation attribute: ${name}`)
-  }
-  return parse(parsedValue, name)
-}
-
 /**
  * Parse an optional mutation attribute as JSON, returning a fallback when the
- * attribute is absent. Use this only for attributes that can be safely
- * defaulted in isolation, without invalidating structural invariants that
- * relate them to other attributes on the same mutation. A present-but-malformed
- * attribute still throws, since that indicates corruption rather than an older
- * schema.
+ * attribute is absent or malformed. Older saved projects may omit attributes
+ * that current serialization always writes, and some projects in the wild
+ * contain attributes whose values are not valid for the expected type.
+ * Returning the fallback in both cases lets the project load rather than
+ * aborting the entire workspace parse.
  * @param xmlElement The mutation element that may contain the attribute.
  * @param name The specific mutation attribute to retrieve and parse.
  * @param parse Validates and narrows the parsed JSON value.
- * @param fallback Value to return when the attribute is absent.
+ * @param fallback Value to return when the attribute is absent or malformed.
  * @returns Parsed and validated mutation attribute value, or `fallback`.
  */
 function parseOptionalMutationJson<T>(
@@ -120,13 +100,12 @@ function parseOptionalMutationJson<T>(
   if (rawValue === null) {
     return fallback
   }
-  let parsedValue: unknown
   try {
-    parsedValue = JSON.parse(rawValue)
+    return parse(JSON.parse(rawValue), name)
   } catch {
-    throw new Error(`Invalid JSON in mutation attribute: ${name}`)
+    console.warn(`Invalid or unexpected mutation attribute "${name}", using default. Raw value: ${rawValue}`)
+    return fallback
   }
-  return parse(parsedValue, name)
 }
 
 /**
@@ -155,6 +134,26 @@ function parseStringArrayMutationValue(value: unknown, name: string): string[] {
   return value
 }
 
+/**
+ * Validate a parsed mutation value as an array of primitives. Argument
+ * defaults from older projects (especially Scratch 2.0 conversions) contain
+ * mixed types — strings for text arguments, numbers for numeric arguments,
+ * and booleans for boolean arguments (e.g. `["",1,1,1,1]`). The old fork
+ * stored these via untyped `JSON.parse` with no validation, and the VM
+ * passes them through as-is.
+ * @param value Parsed mutation value.
+ * @param name Attribute name used in error messages.
+ * @returns Validated array value.
+ */
+function parsePrimitiveArrayMutationValue(value: unknown, name: string): ArgumentDefault[] {
+  const isArgumentDefault = (entry: unknown): entry is ArgumentDefault =>
+    entry === null || typeof entry === 'string' || typeof entry === 'number' || typeof entry === 'boolean'
+  if (!Array.isArray(value) || !value.every(isArgumentDefault)) {
+    throw new Error(`Expected ArgumentDefault[] JSON value in mutation attribute: ${name}`)
+  }
+  return value
+}
+
 /**
  * A drag strategy for the procedures_prototype block that delegates all drag
  * operations to its parent (the procedures_definition block). This lets the
@@ -308,7 +307,7 @@ function callerDomToMutation(this: ProcedureCallBlock, xmlElement: Element) {
   this.procCode_ = getRequiredMutationAttribute(xmlElement, 'proccode')
   const generateshadows = xmlElement.getAttribute('generateshadows')
   this.generateShadows_ = generateshadows !== null ? JSON.parse(generateshadows) === true : false
-  this.argumentIds_ = parseRequiredMutationJson(xmlElement, 'argumentids', parseStringArrayMutationValue)
+  this.argumentIds_ = parseOptionalMutationJson(xmlElement, 'argumentids', parseStringArrayMutationValue, [])
   this.warp_ = parseOptionalMutationJson(xmlElement, 'warp', parseBooleanMutationValue, false)
   this.updateDisplay_()
 }
@@ -349,9 +348,14 @@ function definitionDomToMutation(this: ProcedurePrototypeBlock | ProcedureDeclar
   const prevArgIds = this.argumentIds_
   const prevDisplayNames = this.displayNames_
 
-  this.argumentIds_ = parseRequiredMutationJson(xmlElement, 'argumentids', parseStringArrayMutationValue)
-  this.displayNames_ = parseRequiredMutationJson(xmlElement, 'argumentnames', parseStringArrayMutationValue)
-  this.argumentDefaults_ = parseRequiredMutationJson(xmlElement, 'argumentdefaults', parseStringArrayMutationValue)
+  this.argumentIds_ = parseOptionalMutationJson(xmlElement, 'argumentids', parseStringArrayMutationValue, [])
+  this.displayNames_ = parseOptionalMutationJson(xmlElement, 'argumentnames', parseStringArrayMutationValue, [])
+  this.argumentDefaults_ = parseOptionalMutationJson(
+    xmlElement,
+    'argumentdefaults',
+    parsePrimitiveArrayMutationValue,
+    [],
+  )
 
   // During full XML deserialization (Blockly.Xml.domToWorkspace), the mutation element
   // is part of the parsed XML tree and its parent element also contains <value> children
@@ -1341,7 +1345,7 @@ interface ProcedureBlock extends Blockly.BlockSvg {
 
 export interface ProcedureDeclarationBlock extends ProcedureBlock {
   displayNames_: string[]
-  argumentDefaults_: string[]
+  argumentDefaults_: ArgumentDefault[]
   removeFieldCallback: (field: Blockly.Field) => void
   createArgumentEditor_: (argumentType: ArgumentType, displayName: string) => Blockly.BlockSvg
   focusLastEditor_: () => void
@@ -1361,7 +1365,7 @@ interface ProcedureCallBlock extends ProcedureBlock {
 
 interface ProcedurePrototypeBlock extends ProcedureBlock {
   displayNames_: string[]
-  argumentDefaults_: string[]
+  argumentDefaults_: ArgumentDefault[]
   skipArgumentReporters_: boolean
   createArgumentReporter_: (argumentType: ArgumentType, displayName: string) => Blockly.BlockSvg
   updateArgumentReporterNames_: (prevArgIds: string[], prevDisplayNames: string[]) => void

package/src/fields/scratch_field_dropdown.ts

@@ -7,6 +7,50 @@ import * as Blockly from 'blockly/core'
 class ScratchFieldDropdown extends Blockly.FieldDropdown {
   private originalStyle!: string
 
+  /**
+   * Accept string values even when they are not in the current options list.
+   * Scratch populates some dropdowns from dynamic data (e.g. the sprite list
+   * for motion_goto_menu, motion_pointtowards_menu, sensing_touchingobject_menu)
+   * and deliberately excludes the currently-editing sprite. A block moved into
+   * the sprite it now targets carries a stored value that is therefore absent
+   * from the option list for this editing context, but is still valid — the
+   * runtime reads the stored value directly. Preserve that value so the UI
+   * shows what the block actually does, rather than silently reverting to the
+   * first option.
+   * @param newValue The value to validate.
+   * @returns The value unchanged if it is a string; otherwise null to reject
+   *     the update.
+   */
+  protected override doClassValidation_(newValue?: string): string | null {
+    if (typeof newValue !== 'string') {
+      return null
+    }
+    return newValue
+  }
+
+  /**
+   * When the stored value is not in the current options list, display the raw
+   * value. Blockly's base implementation reads from the last-matched option,
+   * which stays stale when the incoming value does not appear in the list, so
+   * the default would otherwise render as the first option's text.
+   * @returns Text to display for the currently-selected value.
+   */
+  protected override getText_(): string | null {
+    const value = this.getValue()
+    if (value === null) {
+      return super.getText_()
+    }
+    for (const option of this.getOptions(true)) {
+      if (option === 'separator') continue
+      if (option[1] === value) {
+        // Delegate to the base for the matched case so image/HTMLElement
+        // option types render correctly.
+        return super.getText_()
+      }
+    }
+    return value
+  }
+
   showEditor_(event: PointerEvent) {
     super.showEditor_(event)
     const sourceBlock = this.getSourceBlock() as Blockly.BlockSvg