Skip to content

Fix compaction cascade logging for single-provider configs#276

Merged
konard merged 3 commits into
mainfrom
issue-275-0cfd401f04e4
Jul 3, 2026
Merged

Fix compaction cascade logging for single-provider configs#276
konard merged 3 commits into
mainfrom
issue-275-0cfd401f04e4

Conversation

@konard

@konard konard commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Fixes #275

Summary

  • Resolve optional compaction cascade short names with the non-logging resolver so fallback misses do not emit provider-level error logs.
  • Log unresolved default cascade entries at debug level while keeping explicit --compaction-models misses visible as warnings.
  • Add a regression test for a single custom provider config matching the issue shape.

Reproduction

Using the issue's LINK_ASSISTANT_AGENT_CONFIG_CONTENT with only the formalai/formal-ai custom provider, verbose startup previously emitted repeated model not found - refusing to silently fallback error logs while probing the default compaction cascade. After this change, those default fallback misses are debug-level skip logs and no provider error burst is emitted.

Tests

  • bun test ./tests/compaction-model.ts
  • npm test
  • npm run check
  • cargo test --all-features

Adding .gitkeep for PR creation (default mode).
This file will be removed when the task is complete.

Issue: #275
@konard konard self-assigned this Jul 3, 2026
@konard konard changed the title [WIP] Single-provider config logs harmless ProviderModelNotFoundError at error level (compaction-model cascade) Fix compaction cascade logging for single-provider configs Jul 3, 2026
@konard konard marked this pull request as ready for review July 3, 2026 21:08
@konard

konard commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

Working session summary

Implemented and pushed the fix to PR 276: #276

Changed compaction cascade resolution so default fallback misses no longer use the strict provider resolver that logs ProviderModelNotFoundError at error level. Default cascade skips now log at debug; explicit --compaction-models skips still warn. Added a regression test for the single custom provider config from issue 275 and added the required patch changeset.

Verification passed:

  • bun test ./tests/compaction-model.ts
  • npm test
  • npm run check
  • cargo test --all-features
  • node scripts/validate-changeset.mjs
  • GitHub CI JS CI/CD Pipeline passed on head 0b8bc3b

PR is ready for review, merge state is clean, and the local working tree is clean.


This summary was automatically extracted from the AI working session output.

@konard

konard commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

  • Model: GPT-5.5
  • Provider: OpenAI
  • Public pricing estimate: $12.847237

📊 Context and tokens usage:

  • 422.6K / 200K (211%) input tokens, 33.1K / 128K (26%) output tokens

Total: (422.6K + 7.1M cached) input tokens, 33.1K output tokens, $12.847237 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.5
  • Thinking level: max (~31999 tokens)
  • Model: GPT-5.5 (gpt-5.5)

📎 Log file uploaded as Gist (4178KB)


Now working session is ended, feel free to review and add any feedback on the solution draft.

@konard konard merged commit cb57369 into main Jul 3, 2026
15 of 16 checks passed
@konard

konard commented Jul 3, 2026

Copy link
Copy Markdown
Contributor Author

🎉 Auto-merged

This pull request has been automatically merged by hive-mind.

  • All CI checks have passed

Auto-merged by hive-mind with --auto-merge flag

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.

Single-provider config logs harmless ProviderModelNotFoundError at error level (compaction-model cascade)

1 participant