Skip to content

feat: pass searchTerm to generator context - #445

Open
Utkarsh Verma (heyutkarshverma) wants to merge 1 commit into
microsoft:mainfrom
heyutkarshverma:feat/pass-search-term
Open

feat: pass searchTerm to generator context#445
Utkarsh Verma (heyutkarshverma) wants to merge 1 commit into
microsoft:mainfrom
heyutkarshverma:feat/pass-search-term

Conversation

@heyutkarshverma

Copy link
Copy Markdown

This pull request updates how the searchTerm is handled and passed through the generator execution pipeline, ensuring that it is correctly included in the generator context. It also adds a test to verify this behavior.

Generator context and execution improvements:

  • Modified getGeneratorContext in src/runtime/generator.ts to accept and use a searchTerm parameter, ensuring the context includes the correct search term.
  • Updated executeGenerator and executeGeneratorSafely functions to accept and pass the searchTerm parameter through the call chain, ensuring all generator executions receive the correct context. [1] [2] [3]
  • Updated calls to custom generators to pass the new context structure with the correct searchTerm.

Testing improvements:

  • Added a test in src/tests/runtime/generator.test.ts to verify that searchTerm is correctly passed into the generator context.

Copilot AI lite review requested due to automatic review settings August 13, 2026 17:51

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants