[ONCALL-280] fix: requestly collections partially importing in local workspace

[wrongsahil]

Root cause: When header, query params or path params existed without id or isEnabled, it was leading to request getting not imported.

[
                {
                    "key": "Accept",
                    "isEnabled": true,
                    "value": "application/json"
                }
            ],

Summary by CodeRabbit

• Bug Fixes
  • Fixed validation requirements for HTTP headers and query parameters to support optional identifier and enabled status fields, allowing greater flexibility in request configuration.

[coderabbitai]

Walkthrough

This PR modifies the KeyValuePair schema in src/renderer/actions/local-sync/schemas.ts by making two fields optional: the id field and the isEnabled field. Both fields transition from required types to optional types. This change affects the validation and structure of KeyValuePair objects used in HTTP headers, query parameters, and request body structures.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• feat: added description schema #239: Also modifies the KeyValuePair schema by adding an optional description field to the same exported type
• feat: type support schema changes #246: Also modifies the KeyValuePair schema by adding an optional dataType enum field

Suggested reviewers

• nsrCodes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The title accurately describes the main change: fixing requestly collections that were partially importing in local workspace by making KeyValuePair fields optional.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-requestly-collections-not-importing-in-local-fs

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[linear]

ONCALL-280 Requestly JSON importer not importing all endpoints