@happy-nut/monacori 0.1.10 → 0.1.11

package/dist/commands.js

@@ -124,10 +124,12 @@ function launchReviewApp(args) {
     if (args.includes("--no-watch"))
         appArgs.push("--no-watch");
     const electronBinary = resolveElectronBinary();
-    // Tell the user which build is about to run. The app-main path disambiguates a local checkout from
-    // the installed package (their versions can match); printed on the shell that ran `mo`, even when
-    // the app itself is spawned detached.
-    console.error(`monacori: launching ${appMainPath()}`);
+    // In dev only (`npm run dev` sets MONACORI_DEV=1) announce which build is launching, so a local
+    // checkout is distinguishable from the installed package. Normal `mo` runs stay silent — the shell
+    // should be clean, not littered with our internal path.
+    if (process.env.MONACORI_DEV === "1") {
+        console.error(`monacori: launching ${appMainPath()}`);
+    }
     if (args.includes("--foreground")) {
         const result = spawnSync(electronBinary, appArgs, { stdio: "inherit" });
         process.exit(result.status ?? 0);

package/dist/diff.js

@@ -3,8 +3,9 @@ import { existsSync, readFileSync, statSync } from "node:fs";
 import { basename, join } from "node:path";
 import { FLOW_DIR, IMAGE_MAX_BYTES, SOURCE_MAX_FILE_BYTES, SOURCE_MAX_FILES, SOURCE_MAX_TOTAL_BYTES } from "./constants.js";
 import { formatBytes, hashText, isLikelyBinary, languageForPath, stripDiffPath } from "./util.js";
-import { git } from "./git.js";
+import { git, repoRoot } from "./git.js";
 export function readUnifiedDiff(options) {
+    const root = repoRoot();
     const args = ["diff", "--no-ext-diff", "--find-renames", `--unified=${options.context}`];
     if (options.ignoreWhitespace)
         args.push("--ignore-all-space");
@@ -16,7 +17,7 @@ export function readUnifiedDiff(options) {
     }
     args.push("--");
     const result = spawnSync("git", args, {
-        cwd: process.cwd(),
+        cwd: root,
         encoding: "utf8",
         maxBuffer: 1024 * 1024 * 100,
     });
@@ -25,18 +26,18 @@ export function readUnifiedDiff(options) {
     }
     const chunks = [result.stdout ?? ""];
     if (options.includeUntracked && !options.staged) {
-        chunks.push(readUntrackedDiff(options.context));
+        chunks.push(readUntrackedDiff(options.context, root));
     }
     return chunks.filter(Boolean).join("\n");
 }
-function readUntrackedDiff(context) {
-    const files = git(process.cwd(), ["ls-files", "--others", "--exclude-standard"])
+function readUntrackedDiff(context, root) {
+    const files = git(root, ["ls-files", "--others", "--exclude-standard"])
         .split(/\r?\n/)
         .map((line) => line.trim())
         .filter((line) => line && !line.startsWith(`${FLOW_DIR}/`));
     const chunks = [];
     for (const file of files) {
-        const absolute = join(process.cwd(), file);
+        const absolute = join(root, file);
         if (!existsSync(absolute) || !statSync(absolute).isFile()) {
             continue;
         }
@@ -225,14 +226,15 @@ export function collectSourceFiles(diffFiles) {
         }
         changedLinesByPath.set(file.displayPath, nums);
     }
-    const vcsByPath = gitStatusMap(process.cwd());
+    const root = repoRoot();
+    const vcsByPath = gitStatusMap(root);
     for (const file of diffFiles) {
         const kind = vcsByPath.get(file.displayPath);
         if (kind)
             file.vcs = kind; // color the Changes list from the same status map
     }
     const paths = new Set();
-    const gitFiles = git(process.cwd(), ["ls-files", "--cached", "--others", "--exclude-standard"]);
+    const gitFiles = git(root, ["ls-files", "--cached", "--others", "--exclude-standard"]);
     for (const file of gitFiles.split(/\r?\n/)) {
         const path = file.trim();
         if (path && isSourceCandidate(path)) {
@@ -248,7 +250,7 @@ export function collectSourceFiles(diffFiles) {
     let embeddedFiles = 0;
     let embeddedBytes = 0;
     for (const path of Array.from(paths).sort((a, b) => a.localeCompare(b))) {
-        const absolute = join(process.cwd(), path);
+        const absolute = join(root, path);
         const base = {
             path,
             name: basename(path),

package/dist/git.d.ts

@@ -1,4 +1,5 @@
 import type { GitSnapshot } from "./types.js";
 export declare function isGitRepository(root: string): boolean;
 export declare function git(root: string, args: string[]): string;
+export declare function repoRoot(cwd?: string): string;
 export declare function readGitSnapshot(root: string): GitSnapshot;

package/dist/git.js

@@ -13,6 +13,15 @@ export function git(root, args) {
     }
     return (result.stdout ?? "").trim();
 }
+// Resolve the repository root. `git diff` and `git ls-files` print paths relative to it, and the
+// desktop tree shows them as-is — so every filesystem read of those paths must resolve against the
+// SAME root, not process.cwd(). When `mo` runs from a monorepo subdirectory (cwd != root), joining a
+// repo-root-relative path onto cwd points at a file that doesn't exist, which surfaced as a diff with
+// no source preview ("file is not present in the working tree"). Falls back to cwd outside a repo.
+export function repoRoot(cwd = process.cwd()) {
+    const top = git(cwd, ["rev-parse", "--show-toplevel"]);
+    return top || cwd;
+}
 export function readGitSnapshot(root) {
     return {
         branch: git(root, ["branch", "--show-current"]),

package/dist/viewer.css

@@ -532,6 +532,9 @@ summary.tree-focus { background: var(--bg); }
    file's diff body extends all the way down. */
 #diff-view:not(.hidden) { flex: 1 1 auto; display: flex; flex-direction: column; min-height: 0; }
 #diff-view .diff2html-container { flex: 1 1 auto; display: flex; flex-direction: column; min-height: 0; }
+/* Source view mirrors the diff view: fill the content column so the source-body (its last child, after
+   the tabs + toolbar) reaches the bottom even for short files, instead of floating at the top. */
+#source-viewer:not(.hidden) { flex: 1 1 auto; display: flex; flex-direction: column; min-height: 0; }
 .diff2html-container .d2h-wrapper { flex: 1 1 auto; display: flex; flex-direction: column; }
 .diff2html-container .d2h-file-wrapper:last-child { flex: 1 1 auto; }
 .diff2html-container .d2h-file-wrapper:last-child .d2h-files-diff { height: 100%; }
@@ -639,10 +642,13 @@ h1 { margin: 0; font-size: 18px; }
 .source-tab-close:hover { background: var(--line); color: var(--text); }
 .source-body {
   border: 1px solid var(--border);
-  border-radius: 8px;
   overflow: auto;
   background: var(--panel);
   user-select: text;
+  /* Square edges flush to the chrome (no rounded corners), and fill the content column so a short
+     file still reaches the bottom instead of floating at the top — IntelliJ-style. */
+  flex: 1 1 auto;
+  min-height: 0;
 }
 /* Extend the line-number gutter (and its divider) to the bottom of the panel so it never stops at the
    last line with an empty strip + cut-off border below it — one continuous gutter. The .num cell is 58px

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happy-nut/monacori",
-  "version": "0.1.10",
+  "version": "0.1.11",
   "description": "Validation control plane for AI-generated code changes.",
   "type": "module",
   "repository": {