v2.4.0 (#19)

Synogun · Synogun · commit 490eb717124c · 2026-03-03T23:05:03.000-03:00
diff --git a/.github/workflows/deploy_to_pages.yml b/.github/workflows/deploy_to_pages.yml
@@ -1,51 +1,99 @@
-# Simple workflow for deploying static content to GitHub Pages
-name: Deploy static content to Pages
+name: Deploy to GitHub Pages
 
 on:
-    # Runs on pushes targeting the default branch
     push:
         branches: ['main']
-
-    # Allows you to run this workflow manually from the Actions tab
     workflow_dispatch:
 
-# Sets the GITHUB_TOKEN permissions to allow deployment to GitHub Pages
 permissions:
     contents: read
-    pages: write
-    id-token: write
 
-# Allow one concurrent deployment
 concurrency:
     group: 'pages'
-    cancel-in-progress: true
+    cancel-in-progress: false
 
 jobs:
-    # Single deploy job since we're just deploying
-    deploy:
-        environment:
-            name: github-pages
-            url: ${{ steps.deployment.outputs.page_url }}
+    build:
         runs-on: ubuntu-latest
         steps:
             - name: Checkout
-              uses: actions/checkout@v5
+              uses: actions/checkout@v4
+
             - name: Set up Node
               uses: actions/setup-node@v4
               with:
                   node-version: lts/*
                   cache: 'npm'
+
             - name: Install dependencies
               run: npm ci
+
+            - name: Lint
+              run: npm run lint
+
             - name: Build
               run: npm run build
-            - name: Setup Pages
-              uses: actions/configure-pages@v5
+
             - name: Upload artifact
               uses: actions/upload-pages-artifact@v4
               with:
-                  # Upload dist folder
                   path: './dist'
+
+    deploy:
+        needs: build
+        runs-on: ubuntu-latest
+        permissions:
+            pages: write
+            id-token: write
+        environment:
+            name: github-pages
+            url: ${{ steps.deployment.outputs.page_url }}
+        steps:
+            - name: Configure Pages
+              uses: actions/configure-pages@v4
+
             - name: Deploy to GitHub Pages
               id: deployment
               uses: actions/deploy-pages@v4
+
+    release:
+        needs: deploy
+        runs-on: ubuntu-latest
+        permissions:
+            contents: write
+        steps:
+            - name: Checkout
+              uses: actions/checkout@v4
+              with:
+                  fetch-depth: 0
+
+            - name: Get version from package.json
+              id: version
+              run: |
+                  VERSION=$(node -p "require('./package.json').version")
+                  echo "tag=v${VERSION}" >> "$GITHUB_OUTPUT"
+
+            - name: Check if tag already exists
+              id: check_tag
+              run: |
+                  if git rev-parse "refs/tags/${{ steps.version.outputs.tag }}" >/dev/null 2>&1; then
+                      echo "exists=true" >> "$GITHUB_OUTPUT"
+                  else
+                      echo "exists=false" >> "$GITHUB_OUTPUT"
+                  fi
+
+            - name: Extract changelog for version
+              if: steps.check_tag.outputs.exists == 'false'
+              run: |
+                  TAG="${{ steps.version.outputs.tag }}"
+                  awk "/^## \\[${TAG}\\]/{found=1; next} found && /^## \\[v/{exit} found{print}" CHANGELOG.md > release_body.md
+
+            - name: Create GitHub release
+              if: steps.check_tag.outputs.exists == 'false'
+              env:
+                  GH_TOKEN: ${{ github.token }}
+              run: |
+                  gh release create "${{ steps.version.outputs.tag }}" \
+                      --title "${{ steps.version.outputs.tag }}" \
+                      --notes-file release_body.md \
+                      --target "${{ github.sha }}"
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,14 +5,34 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
-## [Unreleased](https://github.com/Synogun/GraphVM/compare/v2.3.0...develop)
+## [Unreleased](https://github.com/Synogun/GraphVM/compare/v2.4.0...develop)
 
 ### Added
 
 ### Changed
 
 ### Fixed
 
+## [v2.4.0](https://github.com/Synogun/GraphVM/compare/v2.3.0...v2.4.0)
+
+### Added
+
+- `Actions` - Added a release step for the project's pipeline.
+- `GraphService` - Added a `resetGraph` function to clear the current graph data and state.
+
+### Changed
+
+- `Algorithms` - Adjusted graph generation algorithms to use `resetGraph` instead of `clearGraph` for better state management.
+
+### Fixed
+
+- `Logger` - Fixed logs being stored in production.
+- `Actions` - Fixed deployment failing but deploying anyway.
+
+### Removed
+
+- `EdgeService` - Removed dead property `numEdges` from the graph's data.
+
 ## [v2.3.0](https://github.com/Synogun/GraphVM/compare/v2.0.0...v2.3.0)
 
 ### Added
diff --git a/package.json b/package.json
@@ -1,7 +1,7 @@
 {
     "name": "graphvm",
     "private": true,
-    "version": "2.3.0",
+    "version": "2.4.0",
     "type": "module",
     "author": "Synogun",
     "license": "MIT",
@@ -11,7 +11,7 @@
         "update-u": "npx ncu --format group -u",
         "dev": "vite",
         "dev:host": "vite --host",
-        "pre:build": "npm run lint",
+        "prebuild": "npm run lint",
         "build": "tsc -b && vite build",
         "lint": "eslint ./src --fix",
         "format": "prettier --write \"src/**/*\"",
diff --git a/src/config/logger.ts b/src/config/logger.ts
@@ -47,6 +47,10 @@ export const Logger = {
      * @param args - additional arguments to include in the log
      */
     _store: (level: string, context: string, message: string, args: unknown[]) => {
+        if (import.meta.env.PROD) {
+            return;
+        }
+
         const timestamp = new Date().toISOString();
         const argsStr = args
             .map((arg) => {
diff --git a/src/hooks/useActionBarLogic.ts b/src/hooks/useActionBarLogic.ts
@@ -3,6 +3,7 @@ import { useGetGraph } from '@/hooks/useGraphRegistry';
 import { addEdges, removeEdges } from '@/services/edgesService';
 import { arrangeGraph, centerGraph } from '@/services/layoutService';
 import { addNode, removeNodes } from '@/services/nodesService';
+import { resetGraph } from '@/services/graphService';
 import { isArrayOfStrings } from '@/types/typeGuards';
 import {
     useGraphProperties,
@@ -58,15 +59,13 @@ export function useActionBarLogic() {
             return;
         }
 
-        graph.elements().remove();
-        graph.data('directed', false);
-        setNodeCount(0);
-        setEdgeCount(0);
-        setSelectedNodes([]);
-        setSelectedEdges([]);
-        graph.data('nodeSelectionOrder', []);
-        graph.data('edgeSelectionOrder', []);
-        setDirected(false);
+        resetGraph(graph, {
+            setNodeCount,
+            setEdgeCount,
+            setSelectedNodes,
+            setSelectedEdges,
+            setDirected,
+        });
     }, [
         graphRef,
         setDirected,
diff --git a/src/services/algorithms/generationAlgorithmsService.ts b/src/services/algorithms/generationAlgorithmsService.ts
@@ -21,6 +21,7 @@ import type {
 } from '@/types/algorithms';
 import type cytoscape from 'cytoscape';
 import { addEdge, addEdges } from '../edgesService';
+import { resetGraph } from '../graphService';
 import { arrangeGraph } from '../layoutService';
 import { addNode } from '../nodesService';
 
@@ -38,7 +39,7 @@ export function generateCompleteGraph(
         );
     }
 
-    clearGraph(graph);
+    resetGraph(graph);
 
     // Add nodes
     for (let i = 0; i < nodeCount; i++) {
@@ -76,7 +77,7 @@ export function generateGridGraph(
         );
     }
 
-    clearGraph(graph);
+    resetGraph(graph);
 
     const totalNodes = rows * cols;
 
@@ -137,7 +138,7 @@ export function generateCircleGraph(
         );
     }
 
-    clearGraph(graph);
+    resetGraph(graph);
 
     // Add nodes
     for (let i = 0; i < nodeCount; i++) {
@@ -178,7 +179,7 @@ export function generateStarGraph(
     }
 
     // Clear existing graph
-    clearGraph(graph);
+    resetGraph(graph);
 
     // Add central node
     const centerNode = addNode(graph);
@@ -215,7 +216,7 @@ export function generateWheelGraph(
         );
     }
 
-    clearGraph(graph);
+    resetGraph(graph);
 
     // Connect outer nodes to center
     const centerNode = addNode(graph);
@@ -268,7 +269,7 @@ export function generateBipartiteGraph(
         );
     }
 
-    clearGraph(graph);
+    resetGraph(graph);
 
     // Add nodes for set A
     const setANodes: cytoscape.NodeSingular[] = [];
@@ -319,7 +320,7 @@ export function generateCompleteBipartiteGraph(
         );
     }
 
-    clearGraph(graph);
+    resetGraph(graph);
 
     // Add nodes for set A
     const setANodes: cytoscape.NodeSingular[] = [];
@@ -369,7 +370,7 @@ export function generateSimpleGraph(
         );
     }
 
-    clearGraph(graph);
+    resetGraph(graph);
 
     // Add nodes
     for (let i = 0; i < nodeCount; i++) {
@@ -406,10 +407,3 @@ export function generateSimpleGraph(
         arrangeGraph(graph, layout);
     }
 }
-
-function clearGraph(graph: cytoscape.Core) {
-    graph.elements().remove();
-    graph.data('directed', false);
-    graph.data('nodeSelectionOrder', []);
-    graph.data('edgeSelectionOrder', []);
-}
diff --git a/src/services/edgesService.ts b/src/services/edgesService.ts
@@ -47,8 +47,6 @@ export function addEdge(
     if (isDirected) {
         core.$id(newId).addClass('directed');
     }
-
-    core.data('numEdges', core.edges().length);
 }
 
 export function addEdges(
diff --git a/src/services/graphService.ts b/src/services/graphService.ts
@@ -3,8 +3,19 @@ import {
     DefaultGraphOptions,
     DefaultNodesData,
 } from '@/constants/graphDefaults';
+import { Logger } from '@Logger';
 import cytoscape from 'cytoscape';
 
+const logger = Logger.createContextLogger('GraphService');
+
+type ResetGraphState = {
+    setNodeCount?: (count: number) => void;
+    setEdgeCount?: (count: number) => void;
+    setSelectedNodes?: (nodes: string[]) => void;
+    setSelectedEdges?: (edges: string[]) => void;
+    setDirected?: (directed: boolean) => void;
+};
+
 export function setGraphDirected(
     core: cytoscape.Core,
     directed: boolean,
@@ -23,6 +34,27 @@ export function setGraphDirected(
     }
 }
 
+export function resetGraph(
+    core: cytoscape.Core,
+    reactState?: ResetGraphState
+): void {
+    logger.info('Resetting graph');
+
+    core.elements().remove();
+    core.data('directed', false);
+    core.data('nodeSelectionOrder', []);
+    core.data('edgeSelectionOrder', []);
+
+    const nodeCount = core.nodes().length;
+    const edgeCount = core.edges().length;
+
+    reactState?.setNodeCount?.(nodeCount);
+    reactState?.setEdgeCount?.(edgeCount);
+    reactState?.setSelectedNodes?.([]);
+    reactState?.setSelectedEdges?.([]);
+    reactState?.setDirected?.(false);
+}
+
 export function newGraph(
     containerId?: string,
     options?: cytoscape.CytoscapeOptions

Original file line number	Diff line number	Diff line change
`@@ -47,8 +47,6 @@ export function addEdge(`
`47`	`47`	`if (isDirected) {`
`48`	`48`	`core.$id(newId).addClass('directed');`
`49`	`49`	`}`
`50`		`-`
`51`		`- core.data('numEdges', core.edges().length);`
`52`	`50`	`}`
`53`	`51`
`54`	`52`	`export function addEdges(`