Skip to content

fix: resolve all test failures from fork restructure#88

Closed
kulvirgit wants to merge 1 commit intorestructure/mainfrom
fix/test-failures
Closed

fix: resolve all test failures from fork restructure#88
kulvirgit wants to merge 1 commit intorestructure/mainfrom
fix/test-failures

Conversation

@kulvirgit
Copy link
Copy Markdown
Collaborator

Summary

  • Fix all 35 test failures + 2 errors from the fork restructure, bringing the suite to 0 failures (1554 pass)
  • Add null guard to NamedError.isInstance() and Azure OpenAI overflow patterns to ProviderError
  • Implement ACP agent synthetic pending events and bash output snapshot streaming with de-duplication
  • Update test references: buildbuilder agent name, relocated module paths, fork package naming, token ratio, config filename

Test plan

  • Full test suite passes: 1554 pass, 0 fail, 5 skip
  • Build passes (bun run build)
  • Typecheck passes (pre-push hook)

Source code fixes:
- Add null guard to `NamedError.isInstance()` to handle null input
- Add Azure OpenAI overflow detection patterns to `ProviderError`
- Implement ACP agent synthetic pending events, bash output snapshot
  streaming with de-duplication

Test fixes:
- Update agent tests: `build` → `builder` + add custom agent expectations
  (analyst, executive, validator, migrator)
- Update test import paths for relocated modules (`bridge/client`,
  `bridge/engine`, `tools/project-scan`)
- Update install tests for fork package naming (`@altimateai/altimate-code-`)
- Fix token estimate test expectations for 3.7 chars/token ratio
- Fix compaction config filename `altimate-code.json` → `opencode.json`

Result: 35 failures → 0 failures, 1554 tests pass.

Co-Authored-By: Kai (Claude Opus 4.6) <noreply@anthropic.com>
@kulvirgit kulvirgit deleted the branch restructure/main March 9, 2026 20:50
@kulvirgit kulvirgit closed this Mar 9, 2026
@kulvirgit kulvirgit deleted the fix/test-failures branch March 9, 2026 20:50
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.

1 participant