Merge pull request #177 from i-VRESSE/dotless-toml-table

Do not treat `[topoaa.mol1]` as a module

CHANGELOG.md

@@ -6,6 +6,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## Unreleased
 
+## @i-vresse/wb-core 3.2.3 - 2024-10-09
+
+### Fixed
+
+- Do not treat `[topoaa.mol1]` as a module
+
 ## @i-vresse/wb-core 3.2.2 - 2024-10-09
 
 ### Added

packages/core/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@i-vresse/wb-core",
-  "version": "3.2.2",
+  "version": "3.2.3",
   "description": "React components to construct a workflow builder application",
   "keywords": [
     "react",

packages/core/src/toml.test.ts

@@ -1112,4 +1112,21 @@ describe('lines2node()', () => {
     const expected = [-1, -1, -1, -1, -1, 0, 0, 1]
     expect(lookup).toEqual(expected)
   })
+
+  it('given 2 sections', () => {
+    const workflow = [
+      '',
+      'molecules = [',
+      ']',
+      '',
+      '[section1]',
+      '',
+      '[section1.mol1]'
+    ].join('\n')
+
+    const lookup = lines2node(workflow)
+
+    const expected = [-1, -1, -1, -1, -1, 0, 0, 0]
+    expect(lookup).toEqual(expected)
+  })
 })

packages/core/src/toml.ts

@@ -309,6 +309,8 @@ export function dedupWorkflow (inp: string): string {
  * For each line in the text, return the node index.
  * -1 for global parameters.
  *
+ * Ignores table with dots in the name.
+ *
  * @param text The TOML text
  * @returns
  */
@@ -317,10 +319,9 @@ export function lines2node (text: string): number[] {
   // highlighter linenumber starts with 1 so add offset
   const nodeLines: number[] = [-1]
   let nodeIndex = -1
-  const isTable = /^\[/
   for (let i = 0; i < lines.length; i++) {
     const line = lines[i]
-    if (isTable.test(line)) {
+    if (line.startsWith('[') && !line.includes('.')) {
       nodeIndex++
     }
     nodeLines.push(nodeIndex)