Public Access
Implement meeting assistant v1
This commit is contained in:
@@ -54,6 +54,16 @@ When in doubt, make the reasoning explicit before editing specs or code.
|
||||
|
||||
A spec/change is not considered done merely because code is merged, tests pass, or `openspec validate` succeeds. A change is ready to be accepted or archived only after the agent has verified the intended behavior through the most direct operational surface available.
|
||||
|
||||
Before archiving any OpenSpec change, perform a refactoring pass over the code and specs touched by that change. The pass must inspect both the current diff and the surrounding implementation context, because a small diff may reveal repeated patterns or structural problems that only become obvious when compared with nearby code.
|
||||
|
||||
Do the refactoring pass in these distinct phases, in order:
|
||||
|
||||
1. Check for DRYness. Look for duplication introduced by the change and for existing nearby duplication that the change now makes worth consolidating. A change may be small on its own, but if it is the fifth copy of the same idea, it is a refactoring target.
|
||||
2. Check for SOLID violations. Look for responsibilities that are mixed together, abstractions that are hard to replace or test, interface shapes that force unrelated dependencies, and code paths that require modifying stable code for each new variant.
|
||||
3. Check whether the implementation can be made simpler under KISS. Remove accidental abstractions, reduce branching, clarify names, and prefer the smallest structure that still supports the tested behavior and current spec.
|
||||
|
||||
Preserve behavior during this pass and run the relevant tests again afterward.
|
||||
|
||||
For Meeting Assistant, prefer verification through:
|
||||
|
||||
- local application endpoints
|
||||
|
||||
Reference in New Issue
Block a user