Commit b60eb31
committed
fix: add missing Required fields (enabled, syntax_error) unblocking CI
CI build 619 failed on two Required<> type exhaustiveness errors:
1. ConsolidationPipeline.ts:66 — DEFAULT_CONSOLIDATION lacked
`enabled` after the prior commit added enabled?: boolean to
ConsolidationConfig. Adding `enabled: true` to defaults makes
Required<ConsolidationConfig> satisfied; behaviour unchanged since
true is already the default behaviour when the field is absent
in user config.
2. ForgeStatsAggregator.ts:61 — rejectionReasons initialiser lacked
the `syntax_error` variant which was added to
ForgeRejectionCategory at some point but the aggregator default
wasn't updated in lockstep. Adds syntax_error: 0 so
Record<ForgeRejectionCategory, number> is exhaustive.
Neither is my introduced bug (#2 is pre-existing); both block the
semantic-release publish pipeline. 216/216 memory tests still pass.1 parent b9f4d65 commit b60eb31
105 files changed
Lines changed: 204 additions & 122 deletions
File tree
- dist
- agents/agency
- api
- runtime
- cognitive_substrate
- personas
- core
- config
- conversation
- llm/providers
- implementations
- streaming
- tools
- permissions
- validation
- discovery
- emergent
- extensions/packs
- media/video
- memory/pipeline
- consolidation
- lifecycle
- nlp/ai_utilities
- orchestration/turn-planner
- provenance/config
- rag
- multimodal
- reranking
- unified
- vector_stores
- safety/guardrails
- structured/output
- vision
- providers
- voice-pipeline
- providers
- web-search
- src
- emergent
- memory/pipeline/consolidation
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
22 | 22 | | |
23 | 23 | | |
24 | 24 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | 1 | | |
2 | 2 | | |
3 | 3 | | |
4 | | - | |
| 4 | + | |
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
| |||
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
| 7 | + | |
8 | 8 | | |
9 | 9 | | |
10 | 10 | | |
| |||
0 commit comments