- To regenerate the JavaScript SDK, run
bun run --cwd packages/sdk/js build. - The default branch in this repo is
dev. - Local
mainref may not exist; usedevororigin/devfor diffs. - Treat
.codex/*.mdplans as execution-tracking documents, not just notes: before starting plan-scoped work, read the referenced plan and respect its currentStatus:. - If the user asks for a plan first, or the task is being tracked in
.codex/, create or update the relevant plan file before implementation starts and give it an explicitStatus:. - Keep the plan lifecycle synchronized with reality: use
planned/pendingbefore execution, switch toin_progressonce implementation actually starts, and leave it non-completed if work is only partially done or verification is still missing. - When implementing a task from a plan under
.codex/, update that plan in the same change flow: adjust scope/details if they changed, record any remaining gaps, and mark the plan'sStatus:ascompletedonly after the task is fully implemented and verified. - When a
.codex/plan is completed and its changes affect the desktop app, runtime, settings, MCP/tools, or other behavior the user is expected to test in the local use-copy, do not stop at code/test completion: finish the same flow withbun run package:win, sync toC:\算法\小应用\Lfcode, and relaunch the use-copy unless the user explicitly says not to sync. - Keep
.codex/*.mdreserved for plans and execution tracking. Store standalone HTML design mockups, visual replicas, and reviewable static UI drafts under.codex/design-html/instead ofoutput/or scattered temp locations. - When creating a design draft in
.codex/design-html/, prefer a self-contained.htmlfile with inline CSS/assets when practical so it can be opened and reviewed directly without extra build steps.
- This repo is a Bun workspace monorepo rooted at
package.json. - Most product code lives under
packages/. - The core runtime and session engine live in
packages/lfcode. - The web UI lives in
packages/app, and the Electron host lives inpackages/desktop. - Shared packages such as
packages/core,packages/ui,packages/plugin,packages/sdk/js,packages/shared, andpackages/scriptsupport the app/runtime layers. - Cross-repo specs live in
specs/. - User-local project config and content under the repo root belong in
.lfcode/(agent/,command/,skills/,themes/,plugins/,tool/,glossary/, plusopencode.jsoncandtui.json). - App settings surfaces now include
packages/app/src/components/settings-skills.tsxandpackages/app/src/components/settings-archives.tsx; skill-management and archive changes should be validated through those settings views, not only through server routes. - Instance-side skill APIs live under
packages/lfcode/src/server/routes/instance/skills.ts; when changing local skill import, discovery, hiding, or deletion flows, keep the app settings UI and this route in sync.
- Keep repository issue investigations and recommended solutions under
issue/. - Use one Markdown file for one problem. Each file must contain
问题,原因,推荐解决方案, and状态sections. - Allowed status values are
未解决,解决中, and已解决. Update the status only when code or runtime verification supports the change. - Issue records describe findings and solution direction; they are not execution plans. Use
.codex/*.mdfor implementation tracking when a task becomes plan-scoped. - Include concrete source paths in issue records when the finding is grounded in repository code. Do not include secrets, tokens, cookies, or full sensitive command output.
- Package management is pinned to Bun (
packageManager: bun@1.3.11); use Bun scripts rather than npm or yarn. - Use Bun workspace commands from the repo root unless a section below says otherwise.
- Main dev entrypoints are
bun run dev,bun run dev:web, andbun run dev:desktop. Rootbun run devsetsLFCODE_HOME=$PWD/.dev-homeand launchespackages/lfcodewith--conditions=browser. - Root validation commands are
bun run lintandbun run typecheck; today those resolve tooxlintandbun turbo typecheck. - The root
bun run testcommand is a guard that intentionally fails; run tests from package directories instead. - From
packages/lfcode, run targeted tests withbun test path/to/test.tsor the package suite withbun test --timeout 30000. - From
packages/app, run unit tests withbun run test:unit. - Do not run
bun run test:e2e,bunx playwright test, or any isolated source/Electron E2E suite. For all desktop or UI automation, first package and sync the current build toC:\算法\小应用\Lfcode, then drive only the running installed use-copy throughbun run app:controlor its installed automation bridge. Do not treat a source dev server or temporary sandbox as product verification. packages/appuseshappydom.tsfor unit tests; when adding narrow UI helpers, prefer package-localbun test --preload ./happydom.ts ./src/...over root-level test commands.- After completing changes in this repo, default to the fast Windows desktop packaging path: run
bun run package:win:fastfrompackages/desktopunless the user explicitly asks for the full slow package flow. - When a task includes pushing changes to GitHub, treat the push as release-facing work by default: update the GitHub release for that pushed version in the same flow, attach the Windows installer produced by
packages/desktop/dist/lfcode-win-x64.exe, and include a concise release summary. Do not stop atgit pushunless the user explicitly says to skip the release update. - For desktop/runtime changes that the user will test in the local installed-use copy, do the development checks, then default to the fast sync flow from
packages/desktop:$env:LFCODE_USE_COPY_DIR='C:\算法\小应用\Lfcode'; bun run sync:win-use-copy:fast; do not stop at dev-server validation. Close the use copy before syncing because running Electron files are locked. WithoutLFCODE_USE_COPY_DIR, the sync script defaults to%USERPROFILE%\.lfcode. - Unless the user explicitly says not to, treat verified desktop/runtime changes as sync-followed work by default: after code changes land, automatically stop
C:\算法\小应用\Lfcode\Lfcode.exe, runsync:win-use-copy:fast, and relaunch the synced use copy instead of asking again whether to sync. - For desktop app testing/debugging, default to the installed use-copy workflow instead of lingering in source
devmode: build or use the latest packaged output, sync toC:\算法\小应用\Lfcode, launchC:\算法\小应用\Lfcode\Lfcode.exe, and reproduce/debug there unless the user explicitly asks for source-mode or CI-only verification. - Unless the user explicitly narrows the scope, apply the same fast package-sync-relaunch rule when closing out a completed
.codex/plan whose result is user-testable in the local use-copy; “plan 完成” itself should be treated as a sync trigger for this repo, not just an internal documentation update. - When the user explicitly asks to sync the installed-use copy, default to the fast flow: stop
C:\算法\小应用\Lfcode\Lfcode.exe, runsync:win-use-copy:fast, then relaunch the synced copy. If Windows leavesLfcode.exeinStatus=Unknownorsync:win-use-copyfails withEACCESonresources\app.asar.unpacked, treat it as a local OS-level stuck-process/lock condition and restart Explorer or Windows before retrying. - Future default behavior in this repo: after each completed code change, automatically use the fast sync chain (
bun run sync:win-use-copy:fast) unless the user explicitly asks not to sync or explicitly asks for the full packaging path. bun run package:winround-trips the current Windows runtime entries listed inpackages/desktop/scripts/preserve-win-runtime.ts:cache,data,state,config.json,lfcode.json,lfcode.jsonc, andopencode.jsonc. Keep new persistent runtime data under those preserved entries, or update the preserve/restore scripts before relying on packaging.- Packaged Windows root config is preserved through
packages/desktop/local-config/lfcode.jsonc;packages/desktop/scripts/sync-local-config.tsrefreshes that template from the currentdist/win-unpacked/lfcode.jsonc, andpackages/desktop/electron-builder.config.tscopies it back into the packaged app root viaextraFiles.
Use a short branch name of at most three words, separated by hyphens. Do not use slashes or type prefixes such as feat/ or fix/.
Examples: session-recovery, fix-scroll-state, regenerate-sdk.
Use conventional commit-style messages and PR titles: type(scope): summary.
Valid types are feat, fix, docs, chore, refactor, and test. Scopes are optional; use the affected package or area when helpful, e.g. core, opencode, tui, app, desktop, sdk, or plugin.
Examples: fix(tui): simplify thinking toggle styling, docs: update contributing guide, chore(sdk): regenerate types.
Follow .github/pull_request_template.md when opening a PR, especially the local verification and UI screenshot/recording sections.
- Keep things in one function unless composable or reusable
- Do not extract single-use helpers preemptively. Inline the logic at the call site unless the helper is reused, hides a genuinely complex boundary, or has a clear independent name that improves the caller.
- Avoid
try/catchwhere possible - Avoid using the
anytype - Use Bun APIs when possible, like
Bun.file() - Rely on type inference when possible; avoid explicit type annotations or interfaces unless necessary for exports or clarity
- Prefer functional array methods (flatMap, filter, map) over for loops; use type guards on filter to maintain type inference downstream
- In
src/config, follow the existing self-export pattern at the top of the file (for exampleexport * as ConfigAgent from "./agent") when adding a new config module.
Reduce total variable count by inlining when a value is only used once.
// Good
const journal = await Bun.file(path.join(dir, "journal.json")).json()
// Bad
const journalPath = path.join(dir, "journal.json")
const journal = await Bun.file(journalPath).json()Avoid unnecessary destructuring. Use dot notation to preserve context.
// Good
obj.a
obj.b
// Bad
const { a, b } = obj- Never alias imports. Do not use
import { foo as bar } from "..."or renamed imports likeresolve as pathResolve. - Never use star imports. Do not use
import * as Foo from "..."orimport type * as Foo from "...". - If a namespace-style value is needed, import the module's own exported namespace by name, for example
import { Project } from "@lfcode-ai/core/project", then referenceProject.ID. - Prefer dynamic imports for heavy modules that are only needed in selected code paths, especially in startup-sensitive entrypoints. Destructure dynamic import bindings near the top of the narrowest scope that needs them so they read like normal imports. Avoid inline chains such as
await import("./module").then((mod) => mod.value())or(await import("./module")).value(). Keep branch-specific imports inside the branch that needs them to preserve lazy loading.
Prefer const over let. Use ternaries or early returns instead of reassignment.
// Good
const foo = condition ? 1 : 2
// Bad
let foo
if (condition) foo = 1
else foo = 2Avoid else statements. Prefer early returns.
// Good
function foo() {
if (condition) return 1
return 2
}
// Bad
function foo() {
if (condition) return 1
else return 2
}When a function has several validation branches or supporting details, make the main function read as the happy path and move supporting details into small helpers below it.
// Good
export function loadThing(input: unknown) {
const config = requireConfig(input)
const metadata = readMetadata(input)
return createThing({ config, metadata })
}
function requireConfig(input: unknown) {
...
}- Keep helpers close to the code they support, below the main export when that improves readability.
- Do not over-abstract simple expressions into many single-use helpers; extract only when it names a real concept like
requireConfigorreadMetadata. - Do not return
Effectfrom helpers unless they actually perform effectful work. Synchronous parsing, validation, and option building should stay synchronous. - Prefer Effect schema helpers such as
Schema.UnknownFromJsonStringandSchema.decodeUnknownOptionover manualJSON.parsewrapped inEffect.trywhen parsing untrusted JSON strings. - Add comments for non-obvious constraints and surprising behavior, not for obvious assignments or control flow.
Use snake_case for field names so column names don't need to be redefined as strings.
// Good
const table = sqliteTable("session", {
id: text().primaryKey(),
project_id: text().notNull(),
created_at: integer().notNull(),
})
// Bad
const table = sqliteTable("session", {
id: text("id").primaryKey(),
projectID: text("project_id").notNull(),
createdAt: integer("created_at").notNull(),
})- Avoid mocks as much as possible
- Test actual implementation, do not duplicate logic into tests
- Tests cannot run from repo root (guard:
do-not-run-tests-from-root); run from package dirs likepackages/lfcode.
- Run package-local type checks with
bun run typecheck; never invoketscdirectly.
- Keep durable prompt admission separate from model execution.
SessionV2.prompt(...)admits one durablesession_inputrow before scheduling advisorySessionExecution.wake(sessionID)unlessresume: falserequests admit-only behavior. The serialized runner promotes admitted inputs into visible user messages at safe boundaries. - Reusing a Session ID adopts the existing Session. Reusing a prompt message ID reconciles an exact retry only when Session, prompt, and delivery mode match; conflicting reuse fails. Historical projected prompts lazily synthesize promoted inbox records during exact retry.
- Keep
SessionExecutionprocess-global and Session-ID based. Its local implementation owns the process-local Session coordinator and discovers placement throughSessionStoreplusLocationServiceMap.get(session.location)only when a drain starts; no layer should take a Session ID. V2 interruption targets the active process-local ownership chain for that Session; idle or missing interruption is a no-op. - Keep
SessionRunner, model resolution, tool registry, permissions, and filesystem Location-scoped. OmittedLocation.workspaceIDmeans implicit-local placement; explicit workspace identity remains reserved for future placement semantics. - Preserve one explicit
llm.stream(request)call per provider turn and reload projected history before durable continuation. Do not bridge through legacySessionPrompt.loop(...)or delegate orchestration to an in-memory tool loop. - Keep local Session drains process-local until clustering is implemented.
SessionRunCoordinatorjoins explicit same-Session resumes, coalesces prompt wakeups, and allows different Sessions to run concurrently. Advisory wakes drain eligible durable inbox rows only; post-crash activity recovery requires a separate explicit design before it may retry provider work. - Keep delivery vocabulary explicit. Prompts steer by default and coalesce into the active activity at the next safe provider-turn boundary. Explicit
queueinputs open FIFO future activities one at a time after the active activity settles. - Keep EventV2 replay owner claims separate from clustered Session execution ownership.
- Keep the System Context algebra, registry, and built-ins in
src/system-context; keep Context Source producers with their observed domains, and keep Session History selection plus Context Epoch persistence Session-owned.