docs(agents): principles table — KISS, DRY, YAGNI, POLA - #69
Merged
Conversation
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Signed-off-by: Ivan Banov <ivanbanov@gmail.com>
ivanbanov
added a commit
that referenced
this pull request
Sep 5, 2026
Column padding was left over from the dropped POLA row (#69); oxfmt --check failed on main because of it. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
ivanbanov
added a commit
that referenced
this pull request
Sep 5, 2026
…ions (#71) * refactor(core): after timers join the send queue, drop pass-through hops - `after` timers dispatch through `enqueue` like any send; the private flush cycle and the unreachable microtask branch in `dispatchAfter` are gone, and the stale check keeps only the entry generation - `send` / `setContext` are class-field arrows; `doSend` and the `notify` wrapper are removed; the action host gets the bound fields directly - one frozen `INIT_EVENT` instead of a fresh boot event per use - `oneOf` picks its branch with a loop instead of `find` Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * perf(core): drop per-event allocations on the send path - runActions / resolve run a single entry directly, no one-item array wrap - resolve builds guard params lazily and takes the action host, so the per-event resolver closure and selectTransition fast path are gone - action host and computed host hold the live ctx/computed objects; no getter call per read (computed proxy reads and stale() checks included) - connector builds the connect() argument once; shallowEqual compares props with two for..in passes, no key arrays Single event +7%, state churn +10%, guard fallthrough unchanged. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * refactor(core): one word for the queue lock — flush `flushing` was the only "flush"; the method, the dev limit, the error text and the spec all said "drain". Align on flush: `flushQueue`, `MAX_FLUSH`. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * chore(docs): reflow the principles table in AGENTS.md Column padding was left over from the dropped POLA row (#69); oxfmt --check failed on main because of it. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds a Principles section at the top of
## CodeinAGENTS.md: a table holding every change to KISS, DRY, YAGNI, and POLA, each with a one-line meaning. Mirrors dunky-dev/ui#55 so the working contract is the same across the dunky repos.🤖 Generated with Claude Code