JSON heuristic: don't consider strings as primitive

[severo]

WDYT @platypii? It's a simple change, but I think it works better.

Screencast.From.2025-10-17.11-07-50.mp4

The only objection might be that an array of very small strings, eg: ['a', 'b'] would be better collapsed than expanded by default. But it would require a bit more complexity in the heuristic, and I'm not sure we would see this kind of data too often

[Copilot]

Pull Request Overview

This PR refines the JSON collapse/expand heuristic by treating strings as non-primitive values, causing arrays and objects containing strings to be expanded by default rather than collapsed. This improves the default viewing experience for string-containing data structures.

Key Changes:

• Modified isPrimitive() to exclude strings from primitive classification
• Updated test descriptions and cases to reflect the new string handling behavior
• Minor CI workflow improvement to restrict push triggers to master branch

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
src/components/Json/helpers.ts	Excludes strings from primitive classification in the isPrimitive() function
src/components/Json/Json.test.tsx	Updates tests to verify new string-handling behavior and adjusts test descriptions
.github/workflows/ci.yml	Restricts push-triggered CI runs to master branch and removes trailing whitespace

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[platypii]

yup looks better! 👍