Merge pull request #4930 from udecode/codex/4111-table-border-wrong-cell

Author: zbeyens

.agents/AGENTS.md

@@ -1,10 +1,8 @@
-- `.agents/AGENTS.md` and `.agents/rules/*.mdc` are source of truth. After editing them, run `install` to sync. Never edit `SKILL.md` directly.
+- `.agents/AGENTS.md` and `.agents/rules/*.mdc` are source of truth. After editing them, run `bun install` to sync. Never edit `SKILL.md` directly.
 - In all interactions and commit messages, be extremely concise and sacrifice grammar for the sake of concision.
 - Answer in English by default. Switch languages only when the user explicitly asks for another language.
 - Prefer the best long-term architecture fix over the nearest local patch. If the real fix is an API or abstraction change, do that.
 
-- Internal agent docs live under `docs/`, not `docs/`. Put solution docs in `docs/solutions/` and plans in `docs/plans/`.
-
 ## Git
 
 - **Git:** Never git add, commit, push, or create PR unless the user explicitly asks, or the active command/skill explicitly requires it.
@@ -21,6 +19,7 @@
 - DX: Optimize for the absolute best developer experience. JSDoc must be first-class for agents. Every API surface should be intuitive for both humans and AI agents.
 - Docs: NEVER write changelog-style language ("has been removed", "new feature", "previously", "now supports"). Docs are user-facing reference for the LATEST state only. Write as if no prior version exists. No migration notes, no "what changed" — just document what IS. Follow docs/solutions/style.md for writing tone/structure.
 - Templates: `templates/**` is CI-controlled output. Never manually edit or commit template source, manifests, or lockfiles during package/app work. If local verification rewrites template files, restore them before handoff. Only change `templates/**` when the user explicitly asks for template work.
+- Barrels: If you change package exports, move public files, add/remove files under exported folders, or CI says `pnpm brl` produced changes, run `pnpm brl` before final verification/commit and include the generated barrel updates.
 - Do not write TDD cases for dead code/legacy removal assertions (for example: "should not contain old API X anymore"). Remove the dead path directly and keep tests focused on current behavior.
 - Prefer inline when used once; extract constants only when reused.
 
@@ -77,8 +76,12 @@ Do not default to `pnpm typecheck` for package work in this repo. Package type c
 
 If filtered package builds still leave unresolved workspace-package imports during typecheck, run `pnpm build` at the repo root before treating the failure as real package debt.
 
+If a local-only build/runtime failure points at corrupted files under `node_modules/.bun` or other non-versioned env state while CI is green, clean local env before changing repo code: remove `node_modules`, relevant app caches like `apps/www/.next` and `apps/www/.contentlayer`, remove `.turbo`, then rerun `pnpm install`.
+
 **CLOSEOUT GATE**: If a task edits code, tests, package manifests, or build/type/lint config, do not post a final handoff until the relevant verification ran in this same turn. If you skip a required check or it fails, say that plainly and do not present the task as done.
 
+If package work changed exports or file layout, run `pnpm brl` before the final verification pass. If `pnpm brl` writes files, keep those barrel updates in the change.
+
 **Required sequence for type checking modified packages:**
 
 1. `pnpm install` - Install all dependencies and update lockfile if needed

.changeset/table-merged-border-fix.md

@@ -0,0 +1,5 @@
+---
+"@platejs/table": patch
+---
+
+Fix merged table border toggles targeting the wrong adjacent cell