ViMax

The error occurs because Pydantic rejects null for required fields typed as int, bool, or str. Make every field that the LLM may return as null Optional and give it a default of None:

from typing import Optional
from pydantic import BaseModel, Field, model_validator

class CameraParentItem(BaseModel):
    parent_cam_idx: Optional[int] = Field(
        default=None,
        description="The index of the parent camera. Use null for the root camera.",
        examples=[0, 1, None],
    )
    parent_shot_idx: Optional[int] = Field(
        default=None,
        description="The index of the dependent shot. Use null for the root shot.",
        examples=[0, 3, None],
    )
    reason: Optional[str] = Field(
        default=None,
        description="The reason for selecting the parent camera.",
    )
    is_parent_fully_covers_child: Optional[bool] = Field(
        default=None,
        description="Whether the parent camera fully covers the child camera's content.",
        examples=[True, False, None],
    )
    missing_info: Optional[str] = Field(
        default=None,
        description="Elements in the child shot not covered by the parent shot. Set to null when fully covered.",
    )

If only the root item may have null parent references, add validation so non-root items remain valid:

@model_validator(mode="after")
def validate_root_item(self):
    if self.parent_cam_idx is None or self.parent_shot_idx is None:
        if self.reason is None or "root" not in self.reason.lower():
            raise ValueError("Only the root camera/shot may have null parent indices")
    return self

Also update the LLM prompt to explicitly state that parent_cam_idx and parent_shot_idx should be null only for the root item; otherwise they must be integers.

Define the missing attributes in Novel2MoviePipeline.__init__. The pipeline uses self.working_dir, self.embeddings, and self.novel_compressor without assigning them. Add an initializer such as:

from pathlib import Path

class Novel2MoviePipeline:
    def __init__(self, embeddings, novel_compressor, working_dir="output", ...):
        self.embeddings = embeddings
        self.novel_compressor = novel_compressor
        self.working_dir = Path(working_dir)
        self.working_dir.mkdir(parents=True, exist_ok=True)

Pass the required dependencies when instantiating the pipeline, and update stale imports to use the current interfaces package. This is fixed by the PR "fix: initialize Novel2MoviePipeline dependencies" for issue #43.