Commit
Β·
b6ec445
1
Parent(s):
c514522
docs: Address all remaining CodeRabbit nitpicks
Browse files- Add language specifier to checklist code block in SPEC
- Grammar fix: "AND" β "and" in P1 bug doc
SPEC_ARCHITECTURAL_DEBT.md
CHANGED
|
@@ -313,7 +313,7 @@ async def test_paid_tier_synthesis():
|
|
| 313 |
|
| 314 |
Before merging ANY refactoring PR:
|
| 315 |
|
| 316 |
-
```
|
| 317 |
[ ] make check # All 318+ unit tests pass
|
| 318 |
[ ] make smoke-free # Free Tier produces real synthesis
|
| 319 |
[ ] make smoke-paid # Paid Tier works (if you have key)
|
|
|
|
| 313 |
|
| 314 |
Before merging ANY refactoring PR:
|
| 315 |
|
| 316 |
+
```text
|
| 317 |
[ ] make check # All 318+ unit tests pass
|
| 318 |
[ ] make smoke-free # Free Tier produces real synthesis
|
| 319 |
[ ] make smoke-paid # Paid Tier works (if you have key)
|
docs/bugs/P1_NO_SYNTHESIS_FREE_TIER.md
CHANGED
|
@@ -45,7 +45,7 @@ The `with_standard_manager()` in Microsoft Agent Framework uses the provided cha
|
|
| 45 |
|
| 46 |
### Secondary Issue: Duplicate Complete Events
|
| 47 |
|
| 48 |
-
Both `MagenticFinalResultEvent`
|
| 49 |
|
| 50 |
---
|
| 51 |
|
|
|
|
| 45 |
|
| 46 |
### Secondary Issue: Duplicate Complete Events
|
| 47 |
|
| 48 |
+
Both `MagenticFinalResultEvent` and `WorkflowOutputEvent` are emitted when the workflow ends. The previous code handled both, yielding "Research complete." twice.
|
| 49 |
|
| 50 |
---
|
| 51 |
|