Tianyi Cui
a96b1e3ae7
Three hard-won lessons from running a 7-PR simplification stack through two waves of review feedback: - New "## Orchestrating review feedback across a stacked PR chain" section: one worktree per branch; a fix belongs on the PR that introduced the issue then flows DOWN; review-fixes are separate commits never amends; delegated work is trust-but-verify (prove a regression guard FAILS on unfixed code); reply in-thread on the merits. - "## Conventions" gains a "Never rewrite a pushed branch" rule next to the merge-commit rule: update a child by merging the parent down, never rebase/amend/force-push a pushed branch; a fix lands on its originating PR. - Extended the "Line coverage is not behavior coverage" defensive-patterns bullet with two corollaries this stack re-taught: (1) a real-load-path test only GUARDS the export shape if a broken shape actually FAILS it — an inject-less composition plugin boots fine on a stray `export default`, so it needs an explicit no-default + unwrapExports assertion; (2) "real entry path" means the PUBLISHED artifact (built lib/bin.js under plain node), not the dev runtime (tsx), which masks boot settle-races, module-resolution differences, and a load failure that loader.await()'s Promise.allSettled swallows.
DeepSeek Harness
Monorepo for the DeepSeek Harness group.
Projects
- DeepSeek Code — DeepSeek's coding agent product.
Development
This monorepo is built on the Cordis framework (vendored as source under vendor/), microkernel-style: everything is a plugin.
pnpm install
pnpm run test # vitest
pnpm run demo:echo # runnable echo-agent example (no API key needed)
pnpm run demo:coding # the real DeepSeek coding agent (needs DEEPSEEK_API_KEY)
For humans, start with the development guide for local setup, hooks, environment variables, and quality gates, then read the architecture design before package work. Local context lives in packages/ and vendor/.
For agents, follow AGENTS.md.