@lightningjs/blits 1.9.0 → 1.9.2

package/CHANGELOG.md

@@ -1,5 +1,18 @@
 # Changelog
 
+## v1.9.2
+
+_5 nov 2024_
+
+- Fixed Element related tests
+- Bumped renderer to version 2.6.0
+
+## v1.9.1
+
+_4 nov 2024_
+
+- Added fix for omitting gap-attribute in Layout component
+
 ## v1.9.0
 
 _4 nov 2024_

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lightningjs/blits",
-  "version": "1.9.0",
+  "version": "1.9.2",
   "description": "Blits: The Lightning 3 App Development Framework",
   "bin": "bin/index.js",
   "exports": {
@@ -9,7 +9,8 @@
     "./transitions": "./src/router/transitions/index.js",
     "./precompiler": "./src/lib/precompiler/precompiler.js",
     "./plugins": "./src/plugins/index.js",
-    "./symbols": "./src/lib/symbols.js"
+    "./symbols": "./src/lib/symbols.js",
+    "./blitsFileConverter": "./src/lib/blitsfileconverter/blitsfileconverter.js"
   },
   "scripts": {
     "test": "c8 npm run test:run",
@@ -49,7 +50,7 @@
   },
   "dependencies": {
     "@lightningjs/msdf-generator": "^1.1.0",
-    "@lightningjs/renderer": "^2.5.1"
+    "@lightningjs/renderer": "^2.6.0"
   },
   "repository": {
     "type": "git",

package/src/engines/L3/element.js

@@ -31,26 +31,27 @@ const layoutFn = function (config) {
   const children = this.children
   const childrenLength = children.length
   let otherDimension = 0
+  const gap = config.gap || 0
   for (let i = 0; i < childrenLength; i++) {
     const node = children[i]
     node[xy] = offset
     // todo: temporary text check, due to 1px width of empty text node
     if (wh === 'width') {
-      offset += node.width + (node.width !== ('text' in node ? 1 : 0) ? config.gap : 0)
+      offset += node.width + (node.width !== ('text' in node ? 1 : 0) ? gap : 0)
     } else if (wh === 'height') {
       offset +=
         'text' in node
           ? node.width > 1
-            ? node.height + config.gap
+            ? node.height + gap
             : 0
           : node.height !== 0
-          ? node.height + config.gap
+          ? node.height + gap
           : 0
     }
     otherDimension = Math.max(otherDimension, node[opositeWh])
   }
   // adjust the size of the layout container
-  this[wh] = offset - (config.gap || 0)
+  this[wh] = offset - gap
   this[opositeWh] = otherDimension
 }
 

package/src/engines/L3/element.test.js

@@ -101,7 +101,7 @@ test('Element - Set `w` property', (assert) => {
 
   assert.equal(el.node['width'], 100, 'Node width parameter should be set')
   assert.equal(el.props.props['width'], 100, 'Props width parameter should be set')
-  assert.equal(el.props.raw.get('w'), 100, "Props' raw map entry should be added")
+  assert.equal(el.props.raw['w'], 100, "Props' raw map entry should be added")
   assert.end()
 })
 
@@ -116,7 +116,7 @@ test('Element - Set `mount` property', (assert) => {
   assert.equal(el.node['mountY'], 20, 'Node mountY parameter should be set')
   assert.equal(el.props.props['mountX'], 10, 'Props mountX parameter should be set')
   assert.equal(el.props.props['mountY'], 20, 'Props mountY parameter should be set')
-  assert.equal(el.props.raw.get('mount'), value, "Props' raw map entry should be added")
+  assert.equal(el.props.raw['mount'], value, "Props' raw map entry should be added")
   assert.end()
 })
 
@@ -128,7 +128,7 @@ test('Element - Set `color` property', (assert) => {
 
   assert.equal(el.node['color'], '0xf0ffffff', 'Node color parameter should be set')
   assert.equal(el.props.props['color'], '0xf0ffffff', 'Props color parameter should be set')
-  assert.equal(el.props.raw.get('color'), 'azure', "Props' raw map entry should be added")
+  assert.equal(el.props.raw['color'], 'azure', "Props' raw map entry should be added")
   assert.end()
 })
 
@@ -141,7 +141,7 @@ test('Element - Set `src` property', (assert) => {
 
   assert.equal(el.node['src'], value, 'Node src parameter should be set')
   assert.equal(el.props.props['src'], value, 'Props src parameter should be set')
-  assert.equal(el.props.raw.get('src'), value, "Props' raw map entry should be added")
+  assert.equal(el.props.raw['src'], value, "Props' raw map entry should be added")
   assert.equal(el.props.props['color'], 0xffffffff, 'Props default color parameter should be set')
   assert.end()
 })
@@ -154,7 +154,7 @@ test('Element - Set `texture` property', (assert) => {
 
   assert.equal(el.node['texture'], 'foo', 'Node texture parameter should be set')
   assert.equal(el.props.props['texture'], 'foo', 'Props texture parameter should be set')
-  assert.equal(el.props.raw.get('texture'), 'foo', "Props' raw map entry should be added")
+  assert.equal(el.props.raw['texture'], 'foo', "Props' raw map entry should be added")
   assert.equal(el.props.props['color'], 0xffffffff, 'Props default color parameter should be set')
   assert.end()
 })
@@ -181,7 +181,7 @@ test('Element - Set `fit` property with a string type value', (assert) => {
     fitVal,
     'Props textureOptions parameter should be set'
   )
-  assert.equal(el.props.raw.get('fit'), fitVal, "Props' raw map entry should be added")
+  assert.equal(el.props.raw['fit'], fitVal, "Props' raw map entry should be added")
   assert.end()
 })
 
@@ -212,7 +212,7 @@ test('Element - Set `fit` property with an object type value', (assert) => {
     1,
     'Node resizeMode "clipY" parameter should be set to correct value'
   )
-  assert.equal(el.props.raw.get('fit'), fitVal, "Props' raw map entry should be added")
+  assert.equal(el.props.raw['fit'], fitVal, "Props' raw map entry should be added")
   assert.end()
 })
 
@@ -243,7 +243,7 @@ test('Element - Set `fit` property with position key as a string type value', (a
     1,
     'Node resizeMode "clipY" parameter should be set to correct value'
   )
-  assert.equal(el.props.raw.get('fit'), fitVal, "Props' raw map entry should be added")
+  assert.equal(el.props.raw['fit'], fitVal, "Props' raw map entry should be added")
   assert.end()
 })
 

package/src/lib/blitsfileconverter/blitsfileconverter.js

@@ -0,0 +1,52 @@
+/*
+ * Copyright 2023 Comcast Cable Communications Management, LLC
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+export default (source) => {
+  const { template, script } = parseBlitsFile(source)
+  return injectTemplate(script, template)
+}
+
+const parseBlitsFile = (source) => {
+  const templateMatch = source.match(/<template>([\s\S]*?)<\/template>/)
+  const scriptMatch = source.match(/<script(?:\s+lang=["'](?:js|ts)["'])?\s*>([\s\S]*?)<\/script>/)
+
+  return {
+    template: templateMatch ? templateMatch[1].trim() : '',
+    script: scriptMatch ? scriptMatch[1].trim() : '',
+  }
+}
+
+const injectTemplate = (script, template) => {
+  const componentRegex =
+    /(Blits\.Component|Component)\s*\(\s*(['"`])(?:(?=(\\?))\3.)*?\2\s*,\s*\{|Blits\.Application\s*\(\s*\{/
+
+  const match = script.match(componentRegex)
+
+  if (!match) {
+    // we might consider initializing a component if it's not found automatically
+    throw new Error(
+      'Could not find Blits.Component, Component, or Blits.Application initialization in the script'
+    )
+  }
+
+  // The insertion point is right after the opening curly brace
+  const insertIndex = match.index + match[0].length
+
+  // Using template literals to preserve multiline strings and escape characters
+  const injection = `\n  template: \`${template.replace(/`/g, '\\`')}\`,\n`
+  return script.slice(0, insertIndex) + injection + script.slice(insertIndex)
+}

package/vite/blitsFileConverter.js

@@ -0,0 +1,38 @@
+/*
+ * Copyright 2023 Comcast Cable Communications Management, LLC
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+import blitsfileconverter from '../src/lib/blitsfileconverter/blitsfileconverter.js'
+
+export default function blitsFileType() {
+  return {
+    name: 'vite-plugin-blits-file-type',
+    enforce: 'pre',
+    transform(src, id) {
+      if (id.endsWith('.blits')) {
+        try {
+          const transformedCode = blitsfileconverter(src)
+          return {
+            code: transformedCode,
+            map: null, // no source map
+          }
+        } catch (error) {
+          this.error(error)
+        }
+      }
+    },
+  }
+}

package/vite/index.js

@@ -17,8 +17,10 @@
 
 import preCompiler from './preCompiler.js'
 import msdfGenerator from './msdfGenerator.js'
+import blitsFileConverter from './blitsFileConverter.js'
 
 export { default as preCompiler } from './preCompiler.js'
 export { default as msdfGenerator } from './msdfGenerator.js'
+export { default as blitsFileConverter } from './blitsFileConverter.js'
 
-export default [preCompiler(), msdfGenerator()]
+export default [blitsFileConverter(), preCompiler(), msdfGenerator()]

package/vite/msdfGenerator.js

@@ -1,3 +1,20 @@
+/*
+ * Copyright 2023 Comcast Cable Communications Management, LLC
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
 import path from 'path'
 import * as fs from 'fs'
 import { createHash } from 'crypto'

package/vite/preCompiler.js

@@ -27,6 +27,13 @@ export default function () {
     },
     transform(source, filePath) {
       if (config.blits && config.blits.precompile === false) return source
+
+      const fileExtension = path.extname(filePath)
+
+      // we should only precompile .js and .ts files
+      if (fileExtension === '.js' || fileExtension === '.ts') {
+        return compiler(source, filePath)
+      }
       const relativePath = path.relative(process.cwd(), filePath)
       return compiler(source, relativePath)
     },