glotfile 0.4.1 → 0.4.2

package/dist/server/cli.js

@@ -1590,6 +1590,7 @@ function buildSystemPrompt(hasPluralItems) {
 function buildBatchPrompt(reqs) {
   const targetLocale = reqs[0]?.targetLocale ?? "";
   const hasPluralItems = reqs.some((r) => r.plural !== void 0);
+  const hasGlossaryItems = reqs.some((r) => r.glossary !== void 0 && r.glossary.length > 0);
   const items = reqs.map((r) => {
     const base = {
       id: r.id,
@@ -1599,7 +1600,7 @@ function buildBatchPrompt(reqs) {
       // Wrap in braces so the model sees "{site}" not "site" — makes the visual
       // connection to the source string obvious and reduces rename errors.
       placeholders: r.placeholders.map((p) => `{${p}}`),
-      glossary: r.glossary ?? [],
+      ...r.glossary?.length ? { glossary: r.glossary } : {},
       hasScreenshot: r.image !== void 0
     };
     if (r.plural) {
@@ -1609,7 +1610,7 @@ function buildBatchPrompt(reqs) {
   });
   const returnFormat = hasPluralItems ? 'For a scalar item (has `source`) return {"id","translation"}; for a plural item (has `plural`) return {"id","forms"} with one string per required category.' : 'Return {"id","translation"} for each item.';
   return `Translate every item below into the target locale: ${targetLocale}. All items share this one target language.
-Glossary entries are constraints you MUST apply. Items with hasScreenshot:true have a screenshot supplied as a separate image block above; use it for context. ${returnFormat} Return JSON {"items":[\u2026]}.
+` + (hasGlossaryItems ? "Glossary entries are constraints you MUST apply. " : "") + `Items with hasScreenshot:true have a screenshot supplied as a separate image block above; use it for context. ${returnFormat} Return JSON {"items":[\u2026]}.
 ` + JSON.stringify(items, null, 2);
 }
 function buildTranslateGemmaSystemPrompt(sourceLocale, targetLocale) {
@@ -4470,7 +4471,7 @@ function createApi(deps) {
         await stream.writeSSE({ event: "error", data: JSON.stringify({ error: e.message }) });
         return;
       }
-      const { skipped } = attachScreenshotsForProvider(reqs, s, projectRoot, provider.supportsVision());
+      const { skipped } = attachScreenshotsForProvider(reqs, s, dirname2(resolve9(deps.statePath)), provider.supportsVision());
       if (skipped) console.warn(`Model "${aiCfg.model}" has no vision support; ${skipped} screenshot(s) ignored.`);
       console.log(`[translate] ${reqs.length} string(s) \u2192 ${aiCfg.model}`);
       let totalWritten = 0;
@@ -4548,7 +4549,7 @@ function createApi(deps) {
       } catch (e) {
         return c.json({ error: e.message }, 400);
       }
-      const { skipped } = attachScreenshotsForProvider(toTranslate, s, projectRoot, provider.supportsVision());
+      const { skipped } = attachScreenshotsForProvider(toTranslate, s, dirname2(resolve9(deps.statePath)), provider.supportsVision());
       if (skipped) console.warn(`Model "${aiCfg.model}" has no vision support; ${skipped} screenshot(s) ignored.`);
       const results = await runLocaleParallel(toTranslate, provider, {}, aiCfg.concurrency);
       const latest = load();
@@ -4777,12 +4778,12 @@ async function readFileResponse(absPath) {
 }
 function buildApp(opts) {
   const app = new Hono2();
-  app.route("/api", createApi({ statePath: opts.statePath, autoExport: true }));
-  const projectRoot = dirname3(resolve10(opts.statePath));
+  const apiDeps = { statePath: opts.statePath, autoExport: true };
+  app.route("/api", createApi(apiDeps));
   app.get("/:dir/*", async (c, next) => {
     const dirSeg = c.req.param("dir");
     if (!dirSeg.endsWith("-screenshots")) return next();
-    const shotsRoot = resolve10(projectRoot, dirSeg);
+    const shotsRoot = resolve10(dirname3(resolve10(apiDeps.statePath)), dirSeg);
     const pathname = decodeURIComponent(new URL(c.req.url).pathname);
     const rest = pathname.slice(`/${dirSeg}`.length);
     const target = resolve10(shotsRoot, "." + rest);

package/dist/server/server.js

@@ -2376,6 +2376,7 @@ function buildSystemPrompt(hasPluralItems) {
 function buildBatchPrompt(reqs) {
   const targetLocale = reqs[0]?.targetLocale ?? "";
   const hasPluralItems = reqs.some((r) => r.plural !== void 0);
+  const hasGlossaryItems = reqs.some((r) => r.glossary !== void 0 && r.glossary.length > 0);
   const items = reqs.map((r) => {
     const base = {
       id: r.id,
@@ -2385,7 +2386,7 @@ function buildBatchPrompt(reqs) {
       // Wrap in braces so the model sees "{site}" not "site" — makes the visual
       // connection to the source string obvious and reduces rename errors.
       placeholders: r.placeholders.map((p) => `{${p}}`),
-      glossary: r.glossary ?? [],
+      ...r.glossary?.length ? { glossary: r.glossary } : {},
       hasScreenshot: r.image !== void 0
     };
     if (r.plural) {
@@ -2395,7 +2396,7 @@ function buildBatchPrompt(reqs) {
   });
   const returnFormat = hasPluralItems ? 'For a scalar item (has `source`) return {"id","translation"}; for a plural item (has `plural`) return {"id","forms"} with one string per required category.' : 'Return {"id","translation"} for each item.';
   return `Translate every item below into the target locale: ${targetLocale}. All items share this one target language.
-Glossary entries are constraints you MUST apply. Items with hasScreenshot:true have a screenshot supplied as a separate image block above; use it for context. ${returnFormat} Return JSON {"items":[\u2026]}.
+` + (hasGlossaryItems ? "Glossary entries are constraints you MUST apply. " : "") + `Items with hasScreenshot:true have a screenshot supplied as a separate image block above; use it for context. ${returnFormat} Return JSON {"items":[\u2026]}.
 ` + JSON.stringify(items, null, 2);
 }
 function buildTranslateGemmaSystemPrompt(sourceLocale, targetLocale) {
@@ -4100,7 +4101,7 @@ function createApi(deps) {
         await stream.writeSSE({ event: "error", data: JSON.stringify({ error: e.message }) });
         return;
       }
-      const { skipped } = attachScreenshotsForProvider(reqs, s, projectRoot, provider.supportsVision());
+      const { skipped } = attachScreenshotsForProvider(reqs, s, dirname2(resolve8(deps.statePath)), provider.supportsVision());
       if (skipped) console.warn(`Model "${aiCfg.model}" has no vision support; ${skipped} screenshot(s) ignored.`);
       console.log(`[translate] ${reqs.length} string(s) \u2192 ${aiCfg.model}`);
       let totalWritten = 0;
@@ -4178,7 +4179,7 @@ function createApi(deps) {
       } catch (e) {
         return c.json({ error: e.message }, 400);
       }
-      const { skipped } = attachScreenshotsForProvider(toTranslate, s, projectRoot, provider.supportsVision());
+      const { skipped } = attachScreenshotsForProvider(toTranslate, s, dirname2(resolve8(deps.statePath)), provider.supportsVision());
       if (skipped) console.warn(`Model "${aiCfg.model}" has no vision support; ${skipped} screenshot(s) ignored.`);
       const results = await runLocaleParallel(toTranslate, provider, {}, aiCfg.concurrency);
       const latest = load();
@@ -4386,12 +4387,12 @@ async function readFileResponse(absPath) {
 }
 function buildApp(opts) {
   const app = new Hono2();
-  app.route("/api", createApi({ statePath: opts.statePath, autoExport: true }));
-  const projectRoot = dirname3(resolve9(opts.statePath));
+  const apiDeps = { statePath: opts.statePath, autoExport: true };
+  app.route("/api", createApi(apiDeps));
   app.get("/:dir/*", async (c, next) => {
     const dirSeg = c.req.param("dir");
     if (!dirSeg.endsWith("-screenshots")) return next();
-    const shotsRoot = resolve9(projectRoot, dirSeg);
+    const shotsRoot = resolve9(dirname3(resolve9(apiDeps.statePath)), dirSeg);
     const pathname = decodeURIComponent(new URL(c.req.url).pathname);
     const rest = pathname.slice(`/${dirSeg}`.length);
     const target = resolve9(shotsRoot, "." + rest);