Fix the logic error of getStartingNodes

[mokeyish]

For the flow graph like below, the original logic for calculating depth is wrong.

  graph LR;
      EndNode-->Y;
      Y-->Z;
      EndNode-->B;
      B-->C;
      C-->D;
      D-->Y;

Y will be skipped early, in fact, it has the second greatest depth.

edit:

Calculation order

  graph LR;
      Configration --> Initialization
      Initialization -- Reuse results--> EndNode
      Initialization --> A
      A --> B
      B --> C
      C --> EndNode

[HenryHengZJ]

im not sure if such flow is allowed, the only way you can split the path is using If Else node, and even after splitting the path, they remain on seperate path and are not allowed to rejoin. do you have a working chatflow that shows this type of flow?

[mokeyish]

The Y&Z, you can think it as the global initializaton/configuration(it's very slow). It would be better to reuse the results and avoid double calculations.

It requires PR #1834.

[mokeyish]

@HenryHengZJ Hi, what are your concerns about this PR? Currently this PR and #1834 are working fine in my local evironment.

[HenryHengZJ]

@HenryHengZJ Hi, what are your concerns about this PR? Currently this PR and #1834 are working fine in my local evironment.

sorry for late getting back. Left a comment on #1834