@speclip/pi-subtitles 0.1.0 → 0.2.0

package/README.md

@@ -9,8 +9,8 @@
 - 输入：`pi-speech` 兼容的词级 JSON 转录，也兼容常见 `{ word, start, end }` 秒级结构。
 - A-roll 映射：按保留源片段压缩时间轴；字幕不会跨越剪辑点。
 - 自动分段：结合标点、停顿、字数、时长和阅读速度生成初稿。
-- Agent 审阅：通过不可变 revision 修正 ASR 文本、标点、断句和换行。
-- 导出：标准 SRT；可选带样式的 ASS；两者都不覆盖已有文件。
+- Agent 审阅：通过不可变 revision 修正 ASR 文本、标点和断句；横屏项目强制单行，审阅阶段也不能绕过。
+- 导出：标准 SRT；可选带横屏/竖屏安全区预设的 ASS；两者都不覆盖已有文件。
 - 溯源：绑定原转录哈希、精确 revision、输出哈希和持久化导出回执。
 
 B-roll 只覆盖画面而保留 A-roll 主音频时，不会改变字幕时间。若后续改变了语速、主音频或 A-roll 片段顺序，应新建字幕项目。
@@ -45,7 +45,8 @@ pi install npm:@speclip/pi-subtitles
 subtitles_create {
   projectId: "launch-captions",
   transcriptPath: "transcripts/launch.json",
-  sourceDurationMs: 93224
+  sourceDurationMs: 93224,
+  layoutPreset: "landscape"
 }
 ```
 
@@ -82,9 +83,9 @@ subtitles_get {
 }
 ```
 
-自动结果只是初稿。Agent 应结合实际音频检查文本、术语、标点、语义分组、阅读速度和两行布局，并继续翻页直到 `hasMore: false`。
+自动结果只是初稿。Agent 应结合实际音频检查文本、术语、标点、语义分组、阅读速度和画幅布局，并继续翻页直到 `hasMore: false`。
 
-默认策略适合中文口播：每行最多 16 个可见字符、最多两行；目标 12–20 字、1.5–4.5 秒、约 9 字/秒。它们用于找更自然的解，不要求每条字幕机械一致。极端时间戳冲突只会局部放宽，不会让整段字幕坍缩为一条或退化成逐字闪烁。
+`layoutPreset` 默认为 `landscape`：横屏字幕严格单行，每条最多 16 个可见字符，目标 8–16 字。`portrait` 允许最多两行。两种画幅都以 1.5–4.5 秒、约 9 字/秒为节奏参考；这些节奏值用于找自然断点，不要求每条机械一致。
 
 ### 3. 写入 Agent 审阅版本
 
@@ -94,7 +95,7 @@ subtitles_apply {
   expectedRevision: 1,
   changeReason: "核对音频后修正产品名和语义标点",
   cues: [
-    { id: "cue-001", beginMs: 320, endMs: 2840, text: "这是修正后的\n第一条字幕。" }
+    { id: "cue-001", beginMs: 320, endMs: 2840, text: "这是修正后的第一条字幕。" }
   ]
 }
 ```
@@ -108,19 +109,13 @@ subtitles_export {
   projectId: "launch-captions",
   revision: 2,
   srtPath: "deliverables/launch.zh-CN.srt",
-  assPath: "deliverables/launch.zh-CN.ass",
-  assStyle: {
-    playResX: 3840,
-    playResY: 2160,
-    fontName: "Noto Sans CJK SC",
-    fontSize: 96,
-    outline: 5,
-    marginV: 180
-  }
+  assPath: "deliverables/launch.zh-CN.ass"
 }
 ```
 
-工具会返回 SRT / ASS 文件引用、SHA-256 和持久化导出回执。默认 ASS 字体是 `Noto Sans CJK SC`；实际渲染环境需要自行安装或嵌入该字体。
+工具会按项目画幅自动使用安全区样式，并返回 SRT / ASS 文件引用、SHA-256 和持久化导出回执。横屏使用固定 `1920×1080` ASS 设计坐标，竖屏使用 `1080×1920`；这与源视频是 1080p 还是 4K 无关，libass 会等比缩放。不要把 4K 源分辨率直接写成 ASS 的 `PlayRes`。
+
+默认样式为大号粗体白字、黑色描边和柔和阴影。横屏位于画面下方安全区并保持单行；竖屏字幕上移，同时扩大右边距避开常见交互区。几何参数只允许在安全范围内微调，不能通过 `assStyle` 把字幕缩回不可读尺寸。默认字体是 `Noto Sans CJK SC`，渲染环境仍需安装该字体或提供可用回退字体。
 
 本包的完成状态是“字幕轨道已生成并校验”，不是“字幕已经烧进视频”。最后应把轨道交给支持字幕的渲染器，并检查实际成片中的同步、遮挡、安全区、字体回退和漏字。
 
@@ -145,7 +140,7 @@ npm run check
 npm pack --dry-run
 ```
 
-GitHub Release 标签必须与 `package.json` 完全一致，例如 `v0.1.0`。发布工作流会在标签快照上重新执行检查并通过 npm OIDC 发布。
+GitHub Release 标签必须与 `package.json` 完全一致，例如 `v0.2.0`。发布工作流会在标签快照上重新执行检查并通过 npm OIDC 发布。
 
 ## License
 

package/extensions/subtitles/index.ts

@@ -49,9 +49,12 @@ const assStyle = Type.Object({
 	fontSize: Type.Optional(Type.Number({ minimum: 1, maximum: 500 })),
 	primaryColour: Type.Optional(Type.String({ pattern: "^&H[0-9A-Fa-f]{8}$" })),
 	outlineColour: Type.Optional(Type.String({ pattern: "^&H[0-9A-Fa-f]{8}$" })),
+	bold: Type.Optional(Type.Boolean()),
 	outline: Type.Optional(Type.Number({ minimum: 0, maximum: 20 })),
 	shadow: Type.Optional(Type.Number({ minimum: 0, maximum: 20 })),
 	alignment: Type.Optional(Type.Integer({ minimum: 1, maximum: 9 })),
+	marginL: Type.Optional(Type.Integer({ minimum: 0, maximum: 4_000 })),
+	marginR: Type.Optional(Type.Integer({ minimum: 0, maximum: 4_000 })),
 	marginV: Type.Optional(Type.Integer({ minimum: 0, maximum: 4_000 })),
 }, { additionalProperties: false });
 
@@ -59,11 +62,15 @@ export default function subtitles(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "subtitles_create",
 		label: "Create subtitle project",
-		description: "Create an immutable workspace subtitle project from a word-timestamp JSON transcript. Optionally remaps retained A-roll source segments onto the final edited timeline, then generates readable cue groups without rendering video.",
+		description: "Create an immutable workspace subtitle project from a word-timestamp JSON transcript. Optionally remaps retained A-roll source segments onto the final edited timeline, then generates readable cue groups. Landscape is the default and strictly remains single-line.",
 		parameters: Type.Object({
 			projectId: Type.String({ minLength: 1, maxLength: 128, pattern: "^[a-z0-9](?:[a-z0-9._-]{0,126}[a-z0-9])?$" }),
 			transcriptPath: Type.String({ minLength: 1, description: "Existing workspace-relative word-timestamp JSON from pi-speech or a compatible ASR." }),
 			sourceDurationMs: Type.Number({ exclusiveMinimum: 0 }),
+			layoutPreset: Type.Optional(Type.Union([
+				Type.Literal("landscape"),
+				Type.Literal("portrait"),
+			], { description: "Canvas layout. Landscape is the default and strictly uses one subtitle line." })),
 			timelineSegments: Type.Optional(Type.Array(timelineSegment, {
 				minItems: 1,
 				maxItems: 1_000,
@@ -128,7 +135,7 @@ export default function subtitles(pi: ExtensionAPI): void {
 	pi.registerTool({
 		name: "subtitles_export",
 		label: "Export subtitle tracks",
-		description: "Export one exact reviewed revision as a new UTF-8 SRT file and optional styled ASS file. Never overwrites outputs and returns a persisted provenance receipt; video burn-in remains an explicit renderer handoff.",
+		description: "Export one exact reviewed revision as a new UTF-8 SRT file and optional styled ASS file using the project's landscape or portrait safe-area preset. Never overwrites outputs and returns a persisted provenance receipt; video burn-in remains an explicit renderer handoff.",
 		parameters: Type.Object({
 			projectId: Type.String({ minLength: 1, maxLength: 128 }),
 			revision: Type.Integer({ minimum: 1 }),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@speclip/pi-subtitles",
-  "version": "0.1.0",
+  "version": "0.2.0",
   "description": "Agent-reviewed subtitle timing, layout, and standard track export for Pi",
   "type": "module",
   "license": "MIT",

package/skills/add-subtitles/SKILL.md

@@ -9,12 +9,12 @@ Use `pi-speech` for word timestamps and this package for final-timeline mapping,
 
 1. Finish the A-roll edit before creating subtitles. Obtain the exact source duration and the ordered retained A-roll source ranges. B-roll overlays do not change subtitle timing while the primary A-roll audio remains unchanged.
 2. Use a `pi-speech`-compatible JSON transcript with word-level `beginMs` and `endMs`. Do not derive cue timing from sentence text, a summary, or B-roll asset timestamps.
-3. Call `subtitles_create` once. Pass `timelineSegments` when A-roll has cuts; omit it only when the final timeline is identical to the source. Each retained range must end on a word-safe edit boundary. The tool rejects a word that straddles a cut.
+3. Call `subtitles_create` once. Set `layoutPreset` to `landscape` or `portrait`; it defaults to `landscape`. Pass `timelineSegments` when A-roll has cuts; omit it only when the final timeline is identical to the source. Each retained range must end on a word-safe edit boundary. The tool rejects a word that straddles a cut.
 4. Treat the generated cues as a proposal. Read every page with `subtitles_get`; do not review only the first page. Check the words against the spoken audio, then check punctuation, semantic grouping, reading speed, line breaks, and whether a cue crosses an editorial thought boundary.
 5. Keep speech meaning intact. Correct ASR spelling or punctuation only with audio evidence. Do not delete content merely to make a line shorter. Preserve intentional fillers, emphasis, terminology, numbers, names, and sentence-ending tone.
-6. Prefer one complete idea per cue, at most two balanced lines, and natural boundaries at strong punctuation or acoustic pauses. Avoid one-character orphan cues and rapid flashes. Do not force every cue to an identical duration; subtitle timing follows the actual delivery rhythm.
+6. Prefer one complete idea per cue and natural boundaries at strong punctuation or acoustic pauses. Landscape captions are always one line and must be split into another timed cue when too long; never insert `\n`. Portrait captions may use at most two balanced lines. Avoid one-character orphan cues and rapid flashes. Do not force every cue to an identical duration; subtitle timing follows the actual delivery rhythm.
 7. If edits are needed, gather the complete cue list across all pages and call `subtitles_apply` with the current `expectedRevision` and a concrete `changeReason`. Never submit one paginated slice as if it were the full revision. If the revision changed, re-read and reconcile instead of overwriting another review.
-8. Call `subtitles_export` with the exact reviewed revision. SRT is the portable timing/text track. Request ASS only when positioning or typography is needed, and choose a font that exists in the render environment. The default ASS style names `Noto Sans CJK SC`; embedding or installing that font is the renderer's responsibility.
+8. Call `subtitles_export` with the exact reviewed revision. SRT is the portable timing/text track. Request ASS for final burn-in: the project layout selects a canonical safe-area preset with large bold white text, black outline, soft shadow, and canvas-independent design coordinates. Do not replace `PlayRes` with the source video's 4K resolution. The default style names `Noto Sans CJK SC`; embedding or installing that font is the renderer's responsibility.
 9. Keep rendering explicit. This package does not burn subtitles into video and does not invent FFmpeg arguments. Hand the returned SRT or ASS artifact and provenance receipt to a renderer that supports subtitle tracks, then visually review the actual final video for safe-area placement, occlusion, glyph fallback, sync, and missing text.
 
 Never overwrite a transcript, subtitle track, or render. If the transcript hash changes, create a new subtitle project so timing decisions stay reproducible.

package/skills/add-subtitles/references/timing-and-layout.md

@@ -10,15 +10,22 @@ final_time = accumulated_retained_duration + source_word_time - range_source_sta
 
 A cut is also a subtitle boundary. Even when the two retained words become adjacent in the final video, a cue must not bridge across the edit because it can expose removed language or create misleading timing.
 
-## Default talking-head policy
+## Talking-head layout policy
 
-- Up to 16 visible characters per line, two lines per cue.
-- Aim for 12–20 visible characters, but preserve semantic grouping first.
+- Landscape: exactly one line per cue, up to 16 visible characters; aim for 8–16.
+- Portrait: up to two balanced lines, 16 visible characters per line; aim for 12–20 total.
 - Aim for 1.5–4.5 seconds; allow 0.833–7 seconds when the speech requires it.
 - Prefer about 9 visible characters per second and relax locally up to 11 before making a worse semantic break.
 - A pause near 250ms is useful weak boundary evidence; 450ms is strong boundary evidence.
 
-These are working defaults, not hard editorial targets. The hard requirements are monotonic non-overlapping timestamps, no cue beyond the final timeline, and preservation of the reviewed spoken content.
+Timing values are working defaults, not hard editorial targets. Landscape single-line layout, monotonic non-overlapping timestamps, no cue beyond the final timeline, and preservation of the reviewed spoken content are hard requirements.
+
+## Safe-area presets
+
+- Landscape uses canonical `1920×1080` ASS coordinates: font 80, bold, outline 3, shadow 3, left/right margins 120, bottom margin 140.
+- Portrait uses canonical `1080×1920` ASS coordinates: font 72, bold, outline 3, shadow 3, left margin 90, right margin 200, bottom margin 460.
+- Keep these design coordinates for 1080p, 2K, and 4K sources. libass scales them to the actual frame.
+- Portrait's larger right and bottom margins avoid common right-side action controls and the lower interaction/caption region.
 
 ## Review order
 

package/src/contracts.ts

@@ -37,6 +37,8 @@ export interface SubtitleCue {
 	text: string;
 }
 
+export type SubtitleLayoutPreset = "landscape" | "portrait";
+
 export interface FileRef {
 	path: string;
 	bytes: number;
@@ -51,9 +53,12 @@ export interface AssStyle {
 	fontSize: number;
 	primaryColour: string;
 	outlineColour: string;
+	bold: boolean;
 	outline: number;
 	shadow: number;
 	alignment: number;
+	marginL: number;
+	marginR: number;
 	marginV: number;
 }
 
@@ -63,6 +68,7 @@ export interface SubtitleProject {
 	createdAt: string;
 	updatedAt: string;
 	currentRevision: number;
+	layoutPreset?: SubtitleLayoutPreset;
 	sourceDurationMs: number;
 	timelineDurationMs: number;
 	transcript: FileRef;

package/src/cues.ts

@@ -130,9 +130,9 @@ function joinTokens(tokens: TimedToken[]): string {
 	return value;
 }
 
-function wrapTokens(tokens: TimedToken[], maxLineChars: number): string {
+function wrapTokens(tokens: TimedToken[], maxLineChars: number, maxLines: number): string {
 	const whole = joinTokens(tokens);
-	if (visibleChars(whole) <= maxLineChars) return whole;
+	if (maxLines === 1 || visibleChars(whole) <= maxLineChars) return whole;
 	for (const lineLimit of [maxLineChars, maxLineChars + 2]) {
 		const candidates: Array<{ cost: number; first: string; second: string }> = [];
 		for (let split = 1; split < tokens.length; split += 1) {
@@ -196,7 +196,7 @@ function selectBoundaries(tokens: TimedToken[], policy: SubtitlePolicy): number[
 		for (let end = start + 1; end <= count; end += 1) {
 			const characters = prefix[end]! - prefix[start]!;
 			const durationMs = Math.max(1, tokens[end - 1]!.endMs - tokens[start]!.beginMs);
-			if (end > start + 1 && (characters > (policy.maxLineChars + 2) * policy.maxLines || durationMs > policy.maxDurationMs)) break;
+			if (end > start + 1 && (characters > policy.maxLineChars * policy.maxLines || durationMs > policy.maxDurationMs)) break;
 			const shortPenalty = durationMs < policy.minDurationMs && !(start === 0 && end === count)
 				? policy.minDurationMs - durationMs : 0;
 			const candidate = costs[start]! + shortPenalty
@@ -222,7 +222,7 @@ function buildGroup(tokens: TimedToken[], policy: SubtitlePolicy): Omit<Subtitle
 		cues.push({
 			beginMs: selected[0]!.beginMs,
 			endMs: Math.max(selected[0]!.beginMs + 1, selected.at(-1)!.endMs),
-			text: wrapTokens(selected, policy.maxLineChars),
+			text: wrapTokens(selected, policy.maxLineChars, policy.maxLines),
 		});
 		start = end;
 	}

package/src/export.ts

@@ -1,4 +1,4 @@
-import type { AssStyle, FileRef, SubtitleCue } from "./contracts.ts";
+import type { AssStyle, FileRef, SubtitleCue, SubtitleLayoutPreset } from "./contracts.ts";
 import { rm } from "node:fs/promises";
 import { resolveExistingWorkspaceFile, writeNewWorkspaceFile } from "./workspace.ts";
 
@@ -6,15 +6,49 @@ export const DEFAULT_ASS_STYLE: AssStyle = {
 	playResX: 1920,
 	playResY: 1080,
 	fontName: "Noto Sans CJK SC",
-	fontSize: 54,
+	fontSize: 80,
 	primaryColour: "&H00FFFFFF",
 	outlineColour: "&H00000000",
+	bold: true,
 	outline: 3,
-	shadow: 0,
+	shadow: 3,
 	alignment: 2,
-	marginV: 90,
+	marginL: 120,
+	marginR: 120,
+	marginV: 140,
 };
 
+export const PORTRAIT_ASS_STYLE: AssStyle = {
+	...DEFAULT_ASS_STYLE,
+	playResX: 1080,
+	playResY: 1920,
+	fontSize: 72,
+	marginL: 90,
+	marginR: 200,
+	marginV: 460,
+};
+
+export function resolveAssStyle(layoutPreset: SubtitleLayoutPreset, overrides: Partial<AssStyle> = {}): AssStyle {
+	const base = layoutPreset === "portrait" ? PORTRAIT_ASS_STYLE : DEFAULT_ASS_STYLE;
+	const expected = `${base.playResX}x${base.playResY}`;
+	if ((overrides.playResX !== undefined && overrides.playResX !== base.playResX)
+		|| (overrides.playResY !== undefined && overrides.playResY !== base.playResY)) {
+		throw new Error(`${layoutPreset} ASS style must use canonical ${expected} design coordinates`);
+	}
+	const resolved = { ...base, ...overrides };
+	const limits = layoutPreset === "portrait"
+		? { font: [60, 90], marginL: [60, 140], marginR: [160, 260], marginV: [430, 500] }
+		: { font: [64, 96], marginL: [90, 180], marginR: [90, 180], marginV: [120, 170] };
+	const within = (value: number, [minimum, maximum]: number[]) => Number.isFinite(value) && value >= minimum! && value <= maximum!;
+	if (!resolved.bold || resolved.alignment !== 2 || !within(resolved.fontSize, limits.font)
+		|| !within(resolved.marginL, limits.marginL) || !within(resolved.marginR, limits.marginR)
+		|| !within(resolved.marginV, limits.marginV) || !within(resolved.outline, [2, 5])
+		|| !within(resolved.shadow, [1, 5])) {
+		throw new Error(`${layoutPreset} ASS style must preserve the readable safe-area preset`);
+	}
+	return resolved;
+}
+
 function pad(value: number, width: number): string {
 	return String(value).padStart(width, "0");
 }
@@ -61,7 +95,7 @@ function assText(value: string): string {
 export function renderAss(cues: SubtitleCue[], style: Partial<AssStyle> = {}): string {
 	assertCues(cues);
 	const resolved = { ...DEFAULT_ASS_STYLE, ...style };
-	const header = `[Script Info]\nScriptType: v4.00+\nPlayResX: ${resolved.playResX}\nPlayResY: ${resolved.playResY}\nWrapStyle: 2\nScaledBorderAndShadow: yes\n\n[V4+ Styles]\nFormat: Name, Fontname, Fontsize, PrimaryColour, SecondaryColour, OutlineColour, BackColour, Bold, Italic, Underline, StrikeOut, ScaleX, ScaleY, Spacing, Angle, BorderStyle, Outline, Shadow, Alignment, MarginL, MarginR, MarginV, Encoding\nStyle: Default,${resolved.fontName},${resolved.fontSize},${resolved.primaryColour},${resolved.primaryColour},${resolved.outlineColour},&H00000000,0,0,0,0,100,100,0,0,1,${resolved.outline},${resolved.shadow},${resolved.alignment},40,40,${resolved.marginV},1\n\n[Events]\nFormat: Layer, Start, End, Style, Name, MarginL, MarginR, MarginV, Effect, Text`;
+	const header = `[Script Info]\nScriptType: v4.00+\nPlayResX: ${resolved.playResX}\nPlayResY: ${resolved.playResY}\nWrapStyle: 2\nScaledBorderAndShadow: yes\n\n[V4+ Styles]\nFormat: Name, Fontname, Fontsize, PrimaryColour, SecondaryColour, OutlineColour, BackColour, Bold, Italic, Underline, StrikeOut, ScaleX, ScaleY, Spacing, Angle, BorderStyle, Outline, Shadow, Alignment, MarginL, MarginR, MarginV, Encoding\nStyle: Default,${resolved.fontName},${resolved.fontSize},${resolved.primaryColour},${resolved.primaryColour},${resolved.outlineColour},&H00000000,${resolved.bold ? -1 : 0},0,0,0,100,100,0,0,1,${resolved.outline},${resolved.shadow},${resolved.alignment},${resolved.marginL},${resolved.marginR},${resolved.marginV},1\n\n[Events]\nFormat: Layer, Start, End, Style, Name, MarginL, MarginR, MarginV, Effect, Text`;
 	const events = cues.map((cue) => `Dialogue: 0,${assTime(cue.beginMs)},${assTime(cue.endMs)},Default,,0,0,0,,${assText(cue.text)}`);
 	return `${header}\n${events.join("\n")}\n`;
 }

package/src/project.ts

@@ -8,11 +8,12 @@ import type {
 	SubtitleExportReceipt,
 	SubtitlePolicy,
 	SubtitleProject,
+	SubtitleLayoutPreset,
 	SubtitleSnapshot,
 	TimelineSegment,
 	TranscriptWord,
 } from "./contracts.ts";
-import { exportSubtitleTracks, renderSrt } from "./export.ts";
+import { exportSubtitleTracks, renderSrt, resolveAssStyle } from "./export.ts";
 import {
 	readWorkspaceJson,
 	resolveExistingWorkspaceFile,
@@ -32,6 +33,7 @@ export interface CreateSubtitleProjectInput {
 	projectId: string;
 	transcriptPath: string;
 	sourceDurationMs: number;
+	layoutPreset?: SubtitleLayoutPreset;
 	timelineSegments?: TimelineSegment[];
 	policy?: Partial<SubtitlePolicy>;
 }
@@ -98,8 +100,14 @@ function validateSourceAndTimeline(sourceDurationMs: number, timelineSegments: T
 	}
 }
 
-function resolvedPolicy(input: Partial<SubtitlePolicy> | undefined): SubtitlePolicy {
-	const policy = { ...DEFAULT_SUBTITLE_POLICY, ...input };
+function resolvedPolicy(layoutPreset: SubtitleLayoutPreset, input: Partial<SubtitlePolicy> | undefined): SubtitlePolicy {
+	if (layoutPreset === "landscape" && input?.maxLines !== undefined && input.maxLines !== 1) {
+		throw new Error("Landscape subtitles must use one line");
+	}
+	const layoutDefaults: Partial<SubtitlePolicy> = layoutPreset === "landscape"
+		? { maxLines: 1, targetMinChars: 8, targetMaxChars: 16 }
+		: { maxLines: 2 };
+	const policy = { ...DEFAULT_SUBTITLE_POLICY, ...layoutDefaults, ...input };
 	for (const [name, value] of Object.entries(policy)) {
 		if (!Number.isFinite(value) || value < 0) throw new Error(`Subtitle policy ${name} must be a finite non-negative number`);
 	}
@@ -148,7 +156,8 @@ export async function createSubtitleProject(cwd: string, input: CreateSubtitlePr
 	const segments = input.timelineSegments ?? [{ id: "a-1", sourceStartMs: 0, sourceEndMs: input.sourceDurationMs }];
 	validateSourceAndTimeline(input.sourceDurationMs, segments);
 	const durationMs = timelineDuration(segments);
-	const policy = resolvedPolicy(input.policy);
+	const layoutPreset = input.layoutPreset ?? "landscape";
+	const policy = resolvedPolicy(layoutPreset, input.policy);
 	const cues = buildSubtitleCues(remapWordsToTimeline(transcriptWords(rawTranscript), segments), durationMs, policy);
 	if (cues.length === 0) throw new Error("Retained A-roll timeline contains no transcribed words");
 	const now = new Date().toISOString();
@@ -158,6 +167,7 @@ export async function createSubtitleProject(cwd: string, input: CreateSubtitlePr
 		createdAt: now,
 		updatedAt: now,
 		currentRevision: 1,
+		layoutPreset,
 		sourceDurationMs: input.sourceDurationMs,
 		timelineDurationMs: durationMs,
 		transcript,
@@ -200,10 +210,20 @@ export async function getSubtitleProject(cwd: string, projectId: string, revisio
 	return { project, snapshot: selectedSnapshot };
 }
 
-function assertReviewedCues(cues: SubtitleCue[], timelineDurationMs: number): void {
+function visibleCharacters(value: string): number {
+	return [...value].filter((character) => !/\s/u.test(character) && !/\p{Cf}/u.test(character)).length;
+}
+
+function assertReviewedCues(cues: SubtitleCue[], project: SubtitleProject): void {
 	if (cues.length === 0 || cues.length > 10_000) throw new Error("cues must contain 1-10000 items");
 	renderSrt(cues);
-	if (cues.at(-1)!.endMs > timelineDurationMs) throw new Error("Subtitle cue exceeds the final timeline duration");
+	if (cues.at(-1)!.endMs > project.timelineDurationMs) throw new Error("Subtitle cue exceeds the final timeline duration");
+	if (project.layoutPreset === "landscape") {
+		if (cues.some((cue) => /[\r\n]/u.test(cue.text))) throw new Error("Landscape subtitles must use one line");
+		if (cues.some((cue) => visibleCharacters(cue.text) > project.policy.maxLineChars)) {
+			throw new Error(`Landscape subtitle exceeds ${project.policy.maxLineChars} visible characters`);
+		}
+	}
 }
 
 async function withProjectLock<T>(cwd: string, projectId: string, action: (directory: string) => Promise<T>): Promise<T> {
@@ -229,7 +249,7 @@ export async function applySubtitleRevision(cwd: string, input: ApplySubtitleRev
 		if (project.currentRevision !== input.expectedRevision) {
 			throw new Error(`Project ${input.projectId} expected revision ${input.expectedRevision} but current revision is ${project.currentRevision}`);
 		}
-		assertReviewedCues(input.cues, project.timelineDurationMs);
+		assertReviewedCues(input.cues, project);
 		const revision = project.currentRevision + 1;
 		const nextSnapshot = snapshot(input.projectId, revision, project.currentRevision, input.cues, input.changeReason.trim());
 		const snapshotPath = join(directory, `snapshots/${revision}.json`);
@@ -264,10 +284,13 @@ export async function exportSubtitleProject(cwd: string, input: {
 }) {
 	const { project, snapshot: selectedSnapshot } = await getSubtitleProject(cwd, input.projectId, input.revision);
 	await assertTranscriptUnchanged(cwd, project);
+	const resolvedAssStyle = input.assPath
+		? resolveAssStyle(project.layoutPreset ?? "landscape", input.assStyle)
+		: undefined;
 	const tracks = await exportSubtitleTracks(cwd, selectedSnapshot.cues, {
 		srtPath: input.srtPath,
 		...(input.assPath ? { assPath: input.assPath } : {}),
-		...(input.assStyle ? { assStyle: input.assStyle } : {}),
+		...(resolvedAssStyle ? { assStyle: resolvedAssStyle } : {}),
 	});
 	const exportId = randomUUID();
 	const receiptPath = projectRelative(input.projectId, `exports/${exportId}.json`);