Fix TypeError: Wait for cytoscape instance to be ready before taking snapshot

[Copilot]

Cypress snapshot test fails when win.cyto is undefined, blocking snapshot generation:

TypeError: Cannot read properties of undefined (reading 'fit')
  at Context.eval (webpack:///./cypress/e2e/e2e/loadDesign.js:53:18)

Changes

• Added proper wait mechanism using cy.window().its('cyto').should('exist') to wait for cytoscape instance
• Leverages Cypress's built-in retry-ability to poll until cytoscape is ready
• Test will now fail explicitly if cytoscape is not available within timeout
• Chained .then() to execute fit() and center() operations once cytoscape is confirmed to exist

// Wait for cytoscape instance to be available and then fit/center
cy.window().its('cyto').should('exist').then((cytoscape) => {
    cytoscape.fit();
    cytoscape.center();
});

This ensures snapshots are only captured when the cytoscape canvas is fully loaded and properly positioned, preventing screenshots from being taken with an incomplete or missing canvas.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Snapshots fail with TypeError: Cannot read properties of undefined (reading 'fit')</issue_title>
<issue_description>### Description

     TypeError: Cannot read properties of undefined (reading 'fit')
      at Context.eval (webpack:///./cypress/e2e/e2e/loadDesign.js:53:18)

---

Contributor Resources and Handbook

The layer5.io website uses Gatsby, React, and GitHub Pages. Site content is found under the master branch.

• 📚 See contributing instructions.
• 🎨 Wireframes and designs for Layer5 site in Figma (open invite)
• 🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack.

Join the Layer5 Community by submitting your community member form.
</issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Snapshots fail with TypeError: Cannot read properties of undefined (reading 'fit') #172

---

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

[leecalcote]

@copilot don't do this. Figure out another way to wait for cytoscape to be ready. There is no point in continuing with a screenshot if cytoscape isn't ready.

[github-actions]

View in catalog, Edit in playground or learn how to interpret Meshery Designs

[Copilot]

Pull request overview

Fixes a Cypress E2E snapshot test failure by guarding viewport operations when the Cytoscape instance isn’t available on window, allowing snapshot generation to proceed.

Changes:

• Add a null check around win.cyto before calling .fit() / .center()
• Log a Cypress runner-visible message when Cytoscape is unavailable

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

captureSnapshot is passed a window argument from the caller, but it calls cy.window() again and uses win instead, leaving the window parameter unused. Consider either using the passed-in window to read window.cyto, or removing the window parameter to avoid confusion and an extra Cypress command.

[Copilot]

@copilot don't do this. Figure out another way to wait for cytoscape to be ready. There is no point in continuing with a screenshot if cytoscape isn't ready.

Updated to wait for cytoscape instance to be ready using cy.window().its('cyto').should('exist'). The test will now properly retry until cytoscape is available or fail with a timeout if it's not ready. Changes in a90489e.