Skip to content

lab to append active operation headers to introspection header#8024

Open
mskorokhodov wants to merge 6 commits into
mainfrom
enhancement/lab-introspection-headers
Open

lab to append active operation headers to introspection header#8024
mskorokhodov wants to merge 6 commits into
mainfrom
enhancement/lab-introspection-headers

Conversation

@mskorokhodov
Copy link
Copy Markdown
Collaborator

@mskorokhodov mskorokhodov commented May 5, 2026

Background

#8023

Description

Introspection now listen for active tab and reusing headers from active tab in own query.

Checklist

  • Input validation
  • Output encoding
  • Authentication management
  • Session management
  • Access control
  • Cryptographic practices
  • Error handling and logging
  • Data protection
  • Communication security
  • System configuration
  • Database security
  • File management
  • Memory management
  • Testing

@mskorokhodov mskorokhodov marked this pull request as draft May 5, 2026 19:34
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces dynamic header support for schema introspection and a throttled fetching mechanism. Feedback identifies that the fetchSchema memoization is unstable, causing the throttle to reset on every keystroke. Additionally, the implementation lacks error handling for header JSON parsing, fails to merge global introspection headers, and omits the necessary UI fields for the new header settings.

Comment thread packages/libraries/laboratory/src/lib/endpoint.ts
Comment thread packages/libraries/laboratory/src/lib/endpoint.ts Outdated
Comment thread packages/libraries/laboratory/src/lib/endpoint.ts
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

🚀 Snapshot Release (alpha)

The latest changes of this PR are available as alpha on npm (based on the declared changesets):

Package Version Info
@graphql-hive/laboratory 0.1.8-alpha-20260513080334-fefd9b638851a3108750ed8fefcc13a498182ff2 npm ↗︎ unpkg ↗︎
@graphql-hive/render-laboratory 0.1.8-alpha-20260513080334-fefd9b638851a3108750ed8fefcc13a498182ff2 npm ↗︎ unpkg ↗︎
hive 11.1.0-alpha-20260513080334-fefd9b638851a3108750ed8fefcc13a498182ff2 npm ↗︎ unpkg ↗︎

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 5, 2026

🐋 This PR was built and pushed to the following Docker images:

Targets: build

Platforms: linux/amd64

Image Tag: fefd9b638851a3108750ed8fefcc13a498182ff2

@mskorokhodov mskorokhodov marked this pull request as ready for review May 7, 2026 09:23
@mskorokhodov mskorokhodov force-pushed the enhancement/lab-introspection-headers branch from 0085f18 to a2fa27f Compare May 12, 2026 21:36
@mskorokhodov mskorokhodov force-pushed the enhancement/lab-introspection-headers branch from a2fa27f to fefd9b6 Compare May 13, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants