chore(deps): update peter-evans/create-pull-request digest to 0979079

.changeset/wise-ends-bake.md

@@ -1,5 +0,0 @@
----
-'mermaid': patch
----
-
-fix(gitgraph): pass gitGraphConfig to renderer functions for applying directives properly.

.github/workflows/e2e-timings.yml

@@ -58,7 +58,7 @@ jobs:
           echo "EOF" >> $GITHUB_OUTPUT
 
       - name: Commit and create pull request
-        uses: peter-evans/create-pull-request@84ae59a2cdc2258d6fa0732dd66352dddae2a412
+        uses: peter-evans/create-pull-request@0979079bc20c05bbbb590a56c21c4e2b1d1f1bbe
         with:
           add-paths: |
             cypress/timings.json

cypress/integration/rendering/gitGraph.spec.js

@@ -1569,514 +1569,4 @@ gitGraph TB:
       {}
     );
   });
-
-  describe('showBranches and showCommitLabel directives', () => {
-    it('77: should show branch lines when showBranches is true (default)', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            commit id: "5"
-            commit id: "6"
-        `,
-        {}
-      );
-    });
-
-    it('78: should hide branch lines when showBranches is false', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: false
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            commit id: "5"
-            commit id: "6"
-        `,
-        {}
-      );
-    });
-
-    it('79: should show commit labels when showCommitLabel is true (default)', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            commit id: "5"
-            commit id: "6"
-        `,
-        {}
-      );
-    });
-
-    it('80: should hide commit labels when showCommitLabel is false', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: false
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            commit id: "5"
-            commit id: "6"
-        `,
-        {}
-      );
-    });
-
-    it('81: should show both branches and commit labels when both directives are true (default)', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-         ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            commit id: "5"
-            commit id: "6"
-        `,
-        {}
-      );
-    });
-
-    it('82: should hide both branches and commit labels when both directives are false', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: false
-              showCommitLabel: false
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            commit id: "5"
-            commit id: "6"
-        `,
-        {}
-      );
-    });
-
-    it('83: should show branch lines with merge commits when showBranches is true', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            merge develop
-            commit id: "5"
-            commit id: "6"
-        `,
-        {}
-      );
-    });
-
-    it('84: should hide branch lines with merge commits when showBranches is false', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: false
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            merge develop
-            commit id: "5"
-            commit id: "6"
-        `,
-        {}
-      );
-    });
-
-    it('85: should show commit labels with tags when showCommitLabel is true', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "1" tag: "v1.0"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3" tag: "v1.1"
-            commit id: "4"
-            checkout main
-            merge develop tag: "v2.0"
-            commit id: "5"
-        `,
-        {}
-      );
-    });
-
-    it('86: should hide commit labels with tags when showCommitLabel is false', () => {
-      imgSnapshotTest(
-        `---
-        config:
-          gitGraph:
-            showBranches: true
-            showCommitLabel: false
-            rotateCommitLabel: false
-            parallelCommits: false
-        ---
-        gitGraph
-          commit id: "1" tag: "v1.0"
-          commit id: "2"
-          branch develop
-          checkout develop
-          commit id: "3" tag: "v1.1"
-          commit id: "4"
-          checkout main
-          merge develop tag: "v2.0"
-          commit id: "5"
-        `,
-        {}
-      );
-    });
-
-    it('87: should show branches with TB orientation when showBranches is true', () => {
-      imgSnapshotTest(
-        `---
-        config:
-          gitGraph:
-            showBranches: true
-            showCommitLabel: true
-            rotateCommitLabel: false
-            parallelCommits: false
-        ---
-        gitGraph TB:
-          commit id: "1"
-          commit id: "2"
-          branch develop
-          checkout develop
-          commit id: "3"
-          commit id: "4"
-          checkout main
-          commit id: "5"
-        `,
-        {}
-      );
-    });
-
-    it('88: should hide branches with TB orientation when showBranches is false', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: false
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph TB:
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            commit id: "5"
-        `,
-        {}
-      );
-    });
-
-    it('89: should show commit labels with BT orientation when showCommitLabel is true', () => {
-      imgSnapshotTest(
-        `---
-        config:
-          gitGraph:
-            showBranches: true
-            showCommitLabel: true
-            rotateCommitLabel: false
-            parallelCommits: false
-        ---
-        gitGraph BT:
-          commit id: "1"
-          commit id: "2"
-          branch develop
-          checkout develop
-          commit id: "3"
-          commit id: "4"
-          checkout main
-          commit id: "5"
-        `,
-        {}
-      );
-    });
-
-    it('90: should hide commit labels with BT orientation when showCommitLabel is false', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: false
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph BT:
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            checkout develop
-            commit id: "3"
-            commit id: "4"
-            checkout main
-            commit id: "5"
-        `,
-        {}
-      );
-    });
-
-    it('91: should render with rotateCommitLabel set to true', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: true
-              rotateCommitLabel: true
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "Alpha"
-            commit id: "Beta"
-            branch develop
-            checkout develop
-            commit id: "Gamma"
-            commit id: "Delta"
-            checkout main
-            commit id: "Epsilon"
-        `,
-        {}
-      );
-    });
-
-    it('92: should render with rotateCommitLabel set to false', () => {
-      imgSnapshotTest(
-        `---
-        config:
-          gitGraph:
-            showBranches: true
-            showCommitLabel: true
-            rotateCommitLabel: false
-            parallelCommits: false
-        ---
-        gitGraph
-          commit id: "Alpha"
-          commit id: "Beta"
-          branch develop
-          checkout develop
-          commit id: "Gamma"
-          commit id: "Delta"
-          checkout main
-          commit id: "Epsilon"
-        `,
-        {}
-      );
-    });
-
-    it('93: should render with parallelCommits set to true', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: true
-          ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            branch feature
-            checkout develop
-            commit id: "3"
-            checkout feature
-            commit id: "4"
-            checkout main
-            commit id: "5"
-            checkout develop
-            commit id: "6"
-            checkout feature
-            commit id: "7"
-        `,
-        {}
-      );
-    });
-
-    it('94: should render with parallelCommits set to false', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-          ---
-          gitGraph
-            commit id: "1"
-            commit id: "2"
-            branch develop
-            branch feature
-            checkout develop
-            commit id: "3"
-            checkout feature
-            commit id: "4"
-            checkout main
-            commit id: "5"
-            checkout develop
-            commit id: "6"
-            checkout feature
-            commit id: "7"
-        `,
-        {}
-      );
-    });
-
-    it('95: should render with custom mainBranchName', () => {
-      imgSnapshotTest(
-        `---
-        config:
-          gitGraph:
-            showBranches: true
-            showCommitLabel: true
-            rotateCommitLabel: false
-            parallelCommits: false
-            mainBranchName: 'trunk'
-        ---
-        gitGraph
-          commit id: "1"
-          commit id: "2"
-          branch develop
-          checkout develop
-          commit id: "3"
-          commit id: "4"
-          checkout trunk
-          commit id: "5"
-          commit id: "6"
-        `,
-        {}
-      );
-    });
-
-    it('96: should render with custom mainBranchOrder', () => {
-      imgSnapshotTest(
-        `---
-          config:
-            gitGraph:
-              showBranches: true
-              showCommitLabel: true
-              rotateCommitLabel: false
-              parallelCommits: false
-              mainBranchOrder: 2
-          ---
-          gitGraph
-            commit id: "1"
-            branch feature1
-            branch feature2
-            checkout feature1
-            commit id: "2"
-            checkout feature2
-            commit id: "3"
-            checkout main
-            commit id: "4"
-        `,
-        {}
-      );
-    });
-  });
 });

packages/mermaid/src/diagrams/git/gitGraph.spec.ts

@@ -1357,35 +1357,4 @@ describe('when parsing a gitGraph', function () {
 
     logWarnSpy.mockRestore();
   });
-
-  describe('gitGraph config directives', () => {
-    it('should expose getConfig method', () => {
-      expect(db.getConfig).toBeDefined();
-      expect(typeof db.getConfig).toBe('function');
-    });
-
-    it('should return config with showBranches property', () => {
-      const config = db.getConfig();
-      expect(config).toBeDefined();
-      expect(config).toHaveProperty('showBranches');
-    });
-
-    it('should return config with showCommitLabel property', () => {
-      const config = db.getConfig();
-      expect(config).toBeDefined();
-      expect(config).toHaveProperty('showCommitLabel');
-    });
-
-    it('should return config with rotateCommitLabel property', () => {
-      const config = db.getConfig();
-      expect(config).toBeDefined();
-      expect(config).toHaveProperty('rotateCommitLabel');
-    });
-
-    it('should return config with parallelCommits property', () => {
-      const config = db.getConfig();
-      expect(config).toBeDefined();
-      expect(config).toHaveProperty('parallelCommits');
-    });
-  });
 });

packages/mermaid/src/diagrams/git/gitGraphRenderer.ts

@@ -1,12 +1,11 @@
 import { select } from 'd3';
-import { setupGraphViewbox } from '../../diagram-api/diagramAPI.js';
+import { getConfig, setupGraphViewbox } from '../../diagram-api/diagramAPI.js';
 import { log } from '../../logger.js';
 import utils from '../../utils.js';
 import type { DrawDefinition } from '../../diagram-api/types.js';
 import type d3 from 'd3';
 import type { Commit, GitGraphDBRenderProvider, DiagramOrientation } from './gitGraphTypes.js';
 import { commitType } from './gitGraphTypes.js';
-import type { GitGraphDiagramConfig } from '../../config.type.js';
 
 interface BranchPosition {
   pos: number;
@@ -22,6 +21,8 @@ interface CommitPositionOffset extends CommitPosition {
   posWithOffset: number;
 }
 
+const DEFAULT_CONFIG = getConfig();
+const DEFAULT_GITGRAPH_CONFIG = DEFAULT_CONFIG?.gitGraph;
 const LAYOUT_OFFSET = 10;
 const COMMIT_STEP = 40;
 const PX = 4;
@@ -286,13 +287,12 @@ const drawCommitLabel = (
   gLabels: d3.Selection<SVGGElement, unknown, HTMLElement, any>,
   commit: Commit,
   commitPosition: CommitPositionOffset,
-  pos: number,
-  gitGraphConfig: GitGraphDiagramConfig
+  pos: number
 ) => {
   if (
     commit.type !== commitType.CHERRY_PICK &&
     ((commit.customId && commit.type === commitType.MERGE) || commit.type !== commitType.MERGE) &&
-    gitGraphConfig.showCommitLabel
+    DEFAULT_GITGRAPH_CONFIG?.showCommitLabel
   ) {
     const wrapper = gLabels.append('g');
     const labelBkg = wrapper.insert('rect').attr('class', 'commit-label-bkg');
@@ -322,7 +322,7 @@ const drawCommitLabel = (
         text.attr('x', commitPosition.posWithOffset - bbox.width / 2);
       }
 
-      if (gitGraphConfig.rotateCommitLabel) {
+      if (DEFAULT_GITGRAPH_CONFIG.rotateCommitLabel) {
         if (dir === 'TB' || dir === 'BT') {
           text.attr(
             'transform',
@@ -514,14 +514,16 @@ const getCommitPosition = (
 const drawCommits = (
   svg: d3.Selection<d3.BaseType, unknown, HTMLElement, any>,
   commits: Map<string, Commit>,
-  modifyGraph: boolean,
-  gitGraphConfig: GitGraphDiagramConfig
+  modifyGraph: boolean
 ) => {
+  if (!DEFAULT_GITGRAPH_CONFIG) {
+    throw new Error('GitGraph config not found');
+  }
   const gBullets = svg.append('g').attr('class', 'commit-bullets');
   const gLabels = svg.append('g').attr('class', 'commit-labels');
   let pos = dir === 'TB' || dir === 'BT' ? defaultPos : 0;
   const keys = [...commits.keys()];
-  const isParallelCommits = gitGraphConfig.parallelCommits ?? false;
+  const isParallelCommits = DEFAULT_GITGRAPH_CONFIG?.parallelCommits ?? false;
 
   const sortKeys = (a: string, b: string) => {
     const seqA = commits.get(a)?.seq;
@@ -553,7 +555,7 @@ const drawCommits = (
       const commitSymbolType = commit.customType ?? commit.type;
       const branchIndex = branchPos.get(commit.branch)?.index ?? 0;
       drawCommitBullet(gBullets, commit, commitPosition, typeClass, branchIndex, commitSymbolType);
-      drawCommitLabel(gLabels, commit, commitPosition, pos, gitGraphConfig);
+      drawCommitLabel(gLabels, commit, commitPosition, pos);
       drawCommitTags(gLabels, commit, commitPosition, pos);
     }
     if (dir === 'TB' || dir === 'BT') {
@@ -810,8 +812,7 @@ const drawArrows = (
 
 const drawBranches = (
   svg: d3.Selection<d3.BaseType, unknown, HTMLElement, any>,
-  branches: { name: string }[],
-  gitGraphConfig: GitGraphDiagramConfig
+  branches: { name: string }[]
 ) => {
   const g = svg.append('g');
   branches.forEach((branch, index) => {
@@ -858,14 +859,14 @@ const drawBranches = (
       .attr('class', 'branchLabelBkg label' + adjustIndexForTheme)
       .attr('rx', 4)
       .attr('ry', 4)
-      .attr('x', -bbox.width - 4 - (gitGraphConfig.rotateCommitLabel === true ? 30 : 0))
+      .attr('x', -bbox.width - 4 - (DEFAULT_GITGRAPH_CONFIG?.rotateCommitLabel === true ? 30 : 0))
       .attr('y', -bbox.height / 2 + 8)
       .attr('width', bbox.width + 18)
       .attr('height', bbox.height + 4);
     label.attr(
       'transform',
       'translate(' +
-        (-bbox.width - 14 - (gitGraphConfig.rotateCommitLabel === true ? 30 : 0)) +
+        (-bbox.width - 14 - (DEFAULT_GITGRAPH_CONFIG?.rotateCommitLabel === true ? 30 : 0)) +
         ', ' +
         (pos - bbox.height / 2 - 1) +
         ')'
@@ -898,13 +899,11 @@ export const draw: DrawDefinition = function (txt, id, ver, diagObj) {
   clear();
 
   log.debug('in gitgraph renderer', txt + '\n', 'id:', id, ver);
-  const db = diagObj.db as GitGraphDBRenderProvider;
-  if (!db.getConfig) {
-    log.error('getConfig method is not available on db');
-    return;
+  if (!DEFAULT_GITGRAPH_CONFIG) {
+    throw new Error('GitGraph config not found');
   }
-  const gitGraphConfig = db.getConfig();
-  const rotateCommitLabel = gitGraphConfig.rotateCommitLabel ?? false;
+  const rotateCommitLabel = DEFAULT_GITGRAPH_CONFIG.rotateCommitLabel ?? false;
+  const db = diagObj.db as GitGraphDBRenderProvider;
   allCommitsDict = db.getCommits();
   const branches = db.getBranchesAsObjArray();
   dir = db.getDirection();
@@ -925,22 +924,27 @@ export const draw: DrawDefinition = function (txt, id, ver, diagObj) {
     g.remove();
   });
 
-  drawCommits(diagram, allCommitsDict, false, gitGraphConfig);
-  if (gitGraphConfig.showBranches) {
-    drawBranches(diagram, branches, gitGraphConfig);
+  drawCommits(diagram, allCommitsDict, false);
+  if (DEFAULT_GITGRAPH_CONFIG.showBranches) {
+    drawBranches(diagram, branches);
   }
   drawArrows(diagram, allCommitsDict);
-  drawCommits(diagram, allCommitsDict, true, gitGraphConfig);
+  drawCommits(diagram, allCommitsDict, true);
 
   utils.insertTitle(
     diagram,
     'gitTitleText',
-    gitGraphConfig.titleTopMargin ?? 0,
+    DEFAULT_GITGRAPH_CONFIG.titleTopMargin ?? 0,
     db.getDiagramTitle()
   );
 
   // Setup the view box and size of the svg element
-  setupGraphViewbox(undefined, diagram, gitGraphConfig.diagramPadding, gitGraphConfig.useMaxWidth);
+  setupGraphViewbox(
+    undefined,
+    diagram,
+    DEFAULT_GITGRAPH_CONFIG.diagramPadding,
+    DEFAULT_GITGRAPH_CONFIG.useMaxWidth
+  );
 };
 
 export default {
@@ -1303,6 +1307,7 @@ if (import.meta.vitest) {
       branchPos.set('main', { pos: 0, index: 0 });
       branchPos.set('develop', { pos: 107.49609375, index: 1 });
       branchPos.set('feature', { pos: 225.70703125, index: 2 });
+      DEFAULT_GITGRAPH_CONFIG!.parallelCommits = true;
       commits.forEach((commit, key) => {
         if (commit.parents.length > 0) {
           curPos = calculateCommitPosition(commit);
@@ -1330,6 +1335,7 @@ if (import.meta.vitest) {
       });
     });
   });
+  DEFAULT_GITGRAPH_CONFIG!.parallelCommits = false;
   it('add', () => {
     commitPos.set('parent1', { x: 1, y: 1 });
     commitPos.set('parent2', { x: 2, y: 2 });