Skip to content

moved readme to contributing ov#2

Merged
Gaurav1998-dev merged 1 commit intomainfrom
readme-update
Sep 5, 2025
Merged

moved readme to contributing ov#2
Gaurav1998-dev merged 1 commit intomainfrom
readme-update

Conversation

@Gaurav1998-dev
Copy link
Copy Markdown

No description provided.

@Gaurav1998-dev Gaurav1998-dev merged commit 9d861c8 into main Sep 5, 2025
0 of 2 checks passed
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Sep 5, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

@robert-inkeep robert-inkeep deleted the readme-update branch September 8, 2025 15:48
@claude claude Bot mentioned this pull request Sep 13, 2025
@pullfrog pullfrog Bot mentioned this pull request Mar 10, 2026
tim-inkeep added a commit that referenced this pull request Apr 15, 2026
…nal messages

- Context window (pullfrog #2, load-bearing): getModelContextWindow() was
  called without args and always returned the 120K default, so the 30%
  oversized threshold was hardcoded at ~36K regardless of the actual
  model. Added currentModelSettings to AgentRunContext, stashed after
  configureModelSettings, and read lazily inside toModelOutput.
- Compression prompt (pullfrog #4, load-bearing): buildCompressPrompt
  only kept role==='system' messages, dropping the original user query
  and conversation-history prefix. Now takes originalMessageCount and
  preserves messages.slice(0, originalMessageCount) as the prefix —
  matching the pre-middleware handlePrepareStepCompression behavior.
- Async-iterator fallback (pullfrog #1): replaced the unsound
  `as unknown as AsyncIterator` cast with a proper Reader → iterator
  adapter so the dead branch is safe if ever triggered.
- Middleware spec-version comment (pullfrog #5): documented which
  @ai-sdk/provider versions the wrapGenerate/wrapStream contract was
  verified against.
- JSON round-trip (pullfrog #3): kept as-is. The round-trip is not a
  no-op — it launders `unknown` tool args through JSONValue and strips
  non-JSON types. Added a comment explaining this.

Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
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