Build and test action (#30)

* Version bump

* Style fixes and test action

.github/workflows/release.yml

@@ -19,7 +19,7 @@ jobs:
           # Defaults to the user or organization that owns the workflow file
           scope: '@traut'
       - run: yarn
-      - run: yarn npm publish
+      - run: yarn publish --verbose --access public
         env:
           NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
   release-github:
@@ -33,6 +33,6 @@ jobs:
           # Defaults to the user or organization that owns the workflow file
           scope: '@traut'
       - run: yarn
-      - run: yarn npm publish
+      - run: yarn publish --verbose --access public
         env:
           NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/test.yml

@@ -0,0 +1,20 @@
+name: Build and test the package
+on:
+  push:
+    branches:
+      - main
+  pull_request:
+    types:
+      - opened
+      - synchronize
+      - reopened
+      - ready_for_review
+  workflow_dispatch:
+
+jobs:
+  build-and-test:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+      - run: yarn
+      - run: yarn lint

eslint.config.mjs

@@ -15,11 +15,11 @@ const compat = new FlatCompat({
 export default [...compat.extends("google"), {
     languageOptions: {
         parser: babelParser,
-        ecmaVersion: 5,
-        sourceType: "script",
-
+        ecmaVersion: 6,
+        sourceType: "module",
         parserOptions: {
             requireConfigFile: false,
+            sourceType: "module",
         },
     },
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "stixview",
-  "version": "2.1.3a1",
+  "version": "2.2.0",
   "description": "Embeddable STIX2 graph viewer in JS",
   "main": "src/index.js",
   "repository": {

src/viewer.js

@@ -14,12 +14,7 @@ import {readFile} from './utils.js';
 import {bundleToGraphElements} from './data.js';
 import {renderSidebarContent} from './sidebar.js';
 
-import {
-  computePosition,
-  flip,
-  shift,
-  limitShift,
-} from '@floating-ui/dom';
+import {computePosition} from '@floating-ui/dom';
 
 
 cytoscape.use(klay);
@@ -32,27 +27,16 @@ cytoscape.use(cise);
 // cytoscape-popper requires an explicit factory
 // https://github.com/cytoscape/cytoscape.js-popper?tab=readme-ov-file#migration-from-v2
 function popperFactory(ref, content, opts) {
-   // see https://floating-ui.com/docs/computePosition#options
-   const popperOptions = {
-       // matching the default behaviour from Popper@2
-       // https://floating-ui.com/docs/migration#configure-middleware
-       middleware: [
-           flip(),
-           shift({limiter: limitShift()})
-       ],
-       ...opts,
-   }
-
-   function update() {
-       computePosition(ref, content, opts).then(({x, y}) => {
-           Object.assign(content.style, {
-               left: `${x}px`,
-               top: `${y}px`,
-           });
-       });
-   }
-   update();
-   return { update };
+    function update() {
+        computePosition(ref, content, opts).then(({x, y}) => {
+            Object.assign(content.style, {
+                left: `${x}px`,
+                top: `${y}px`,
+            });
+        });
+    }
+    update();
+    return {update};
 }
 
 cytoscape.use(cytoscapePopper(popperFactory));
@@ -257,15 +241,15 @@ function clustersForCise(cy) {
     // https://github.com/iVis-at-Bilkent/cytoscape.js-cise/
 
     const options = {};
-    let clusters = cy.elements().markovClustering(options);
+    const clusters = cy.elements().markovClustering(options);
 
-    for(var i = 0; i < clusters.length; i++){
-        for(var j = 0; j < clusters[i].length; j++){
+    for (let i = 0; i < clusters.length; i++) {
+        for (let j = 0; j < clusters[i].length; j++) {
             clusters[i][j]._private.data.clusterID = i;
         }
     };
 
-    let arrayOfClusterArrays = [];
+    const arrayOfClusterArrays = [];
     cy.nodes().forEach(function (node) {
         arrayOfClusterArrays.push([node.data('id')]);
     });
@@ -274,9 +258,8 @@ function clustersForCise(cy) {
 
 
 function runLayout(cy, layoutName) {
-
     const localProps = {};
-    if (layoutName === "cise") {
+    if (layoutName === 'cise') {
         localProps.clusters = clustersForCise(cy);
     }