@nodaro/shared 2.17.0 → 2.18.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nodaro/shared",
-  "version": "2.17.0",
+  "version": "2.18.0",
   "description": "Shared types, model catalog, wire contracts, and structural vocabularies for the Nodaro platform and SDK.",
   "type": "module",
   "license": "Apache-2.0",

package/src/__tests__/apply-video-negative-prompt.test.ts

@@ -1,5 +1,37 @@
 import { describe, it, expect } from "vitest"
-import { applyVideoNegativePrompt, NATIVE_NEGATIVE_VIDEO_PROVIDERS } from "../model-constants.js"
+import {
+  applyVideoNegativePrompt,
+  videoNegativeSuffix,
+  NATIVE_NEGATIVE_VIDEO_PROVIDERS,
+} from "../model-constants.js"
+
+/**
+ * A composed prompt that ends with the `[style]` section — what
+ * `@nodaro/prompts` emits for any run that picks a look dimension. The section
+ * has no terminator, so a `"\n"`-joined `Avoid:` reads as one more look clause
+ * under its header.
+ */
+const SECTIONED = "a knight rides\n\n[style]:\nanime style\nwide shot"
+
+describe("videoNegativeSuffix", () => {
+  it("joins on one newline for an ordinary prompt", () => {
+    expect(videoNegativeSuffix("blurry", "a knight rides")).toBe("\nAvoid: blurry")
+  })
+
+  it("closes an open `[style]` section with a blank line", () => {
+    expect(videoNegativeSuffix("blurry", SECTIONED)).toBe("\n\nAvoid: blurry")
+  })
+
+  it("joins on one newline once something else closed the section", () => {
+    expect(videoNegativeSuffix("blurry", `${SECTIONED}\n\nthe person from @image_1`))
+      .toBe("\nAvoid: blurry")
+  })
+
+  it("is the WIDEST form with no base — what a caller reserving room must budget", () => {
+    // `effectiveVideoPromptCeiling` reserves before the prompt exists.
+    expect(videoNegativeSuffix("blurry")).toBe("\n\nAvoid: blurry")
+  })
+})
 
 describe("applyVideoNegativePrompt", () => {
   it("returns prompt unchanged + nativeNegativePrompt when provider is native", () => {
@@ -14,6 +46,11 @@ describe("applyVideoNegativePrompt", () => {
     expect(result.nativeNegativePrompt).toBeUndefined()
   })
 
+  it("separates 'Avoid: …' from an unterminated `[style]` section", () => {
+    const result = applyVideoNegativePrompt(SECTIONED, "blurry", "wan-animate-move")
+    expect(result.prompt).toBe(`${SECTIONED}\n\nAvoid: blurry`)
+  })
+
   it("returns 'Avoid: …' alone when prompt is empty and provider is non-native", () => {
     const result = applyVideoNegativePrompt("", "blurry", "veo3")
     expect(result.prompt).toBe("Avoid: blurry")

package/src/index.ts

@@ -38,6 +38,7 @@ export {
   NATIVE_NEGATIVE_PROMPT_MODELS,
   NATIVE_NEGATIVE_VIDEO_PROVIDERS,
   applyVideoNegativePrompt,
+  videoNegativeSuffix,
   MODELS_WITH_REFERENCE_IMAGE_SUPPORT,
   T2I_TO_I2I_VARIANT,
   REF_IMAGE_MAX_LIMITS,

package/src/model-constants.ts

@@ -295,6 +295,35 @@ export const NATIVE_NEGATIVE_VIDEO_PROVIDERS = new Set<string>([
   "wan-s2v",
 ])
 
+/**
+ * The `[style]` section header, verbatim. A composed prompt that ends with that
+ * section has NO terminator (`@nodaro/prompts`' `prompt-style-section.ts` owns
+ * the shape and the reasoning), so a `"\n"`-joined `Avoid:` would read as one
+ * more look clause under its header. Duplicated rather than imported because
+ * `@nodaro/prompts` depends on THIS package and never the reverse; the two are
+ * pinned together behaviourally by that package's
+ * `__tests__/style-section-boundary.test.ts`, which runs the real composer
+ * through this clamp.
+ */
+const STYLE_SECTION_HEADER_LINE = "[style]:\n"
+
+/**
+ * The `Avoid: …` suffix `applyVideoNegativePrompt` folds into a prompt, with the
+ * separator the base prompt calls for: a blank line when the base ends INSIDE an
+ * unterminated `[style]` section, a single newline otherwise.
+ *
+ * `base` omitted = the WIDEST form, which is what a caller reserving room before
+ * the prompt exists must budget for (`effectiveVideoPromptCeiling`).
+ */
+export function videoNegativeSuffix(negativePrompt: string, base?: string): string {
+  const gapAt = base?.lastIndexOf("\n\n") ?? -1
+  const lastBlock = base === undefined ? undefined : (gapAt >= 0 ? base.slice(gapAt + 2) : base)
+  const separator = lastBlock === undefined || lastBlock.startsWith(STYLE_SECTION_HEADER_LINE)
+    ? "\n\n"
+    : "\n"
+  return `${separator}Avoid: ${negativePrompt}`
+}
+
 /**
  * Apply the negative prompt to a video-provider request.
  *
@@ -306,6 +335,8 @@ export const NATIVE_NEGATIVE_VIDEO_PROVIDERS = new Set<string>([
  * Otherwise, the helper appends `Avoid: <negativePrompt>` to the prompt
  * (consistent with `buildImagePrompt` in `prompt-builder.ts`) so the model
  * still sees the negative intent, and returns `nativeNegativePrompt = undefined`.
+ * The line is separated by {@link videoNegativeSuffix}, which closes an open
+ * `[style]` section with a blank line rather than reading as a look clause.
  *
  * Empty / missing negative is a no-op.
  *
@@ -338,7 +369,10 @@ export function applyVideoNegativePrompt(
   // assembler), then clamp the whole to the model cap.
   const base = prompt && prompt.trim().length > 0 ? prompt : ""
   if (!base) return { prompt: `Avoid: ${neg}`.slice(0, promptMax), nativeNegativePrompt: undefined }
-  const avoid = `\nAvoid: ${neg}`
+  // The separator is decided on the WHOLE base: the clamp below can only cut the
+  // tail, and a cut either leaves the `[style]` section behind or lands inside
+  // it, so the suffix never needs to grow after the reservation.
+  const avoid = videoNegativeSuffix(neg, base)
   const room = Math.max(0, promptMax - avoid.length)
   const injected = `${base.slice(0, room)}${avoid}`.slice(0, promptMax)
   return { prompt: injected, nativeNegativePrompt: undefined }