chore: rename package and reorder inputs, update model defaults

[alwaysmavs]

No description provided.

[coderabbitai]

Walkthrough

Reorders the public Inputs TypedDict field translated_file, moves the translated_file input into a new collapsed "Translate" group in the task manifest, updates LLM model parameters (top_p: 1, max_tokens: 8100), and renames package name from books_translator to books-translator.

Changes

Cohort / File(s)	Summary
TypedDict field reorder tasks/translator/__init__.py	Moved `translated_file: str
Input grouping & LLM tuning tasks/translator/task.oo.yaml	Removed top-level translated_file input; added a new collapsed Translate group containing nullable translated_file (string, save widget, epub filter); updated model block top_p: 1 and max_tokens: 8100; minor write_mode description quoting change.
Package metadata package.oo.yaml	Changed package name from books_translator → books-translator.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• Verify alignment between Inputs TypedDict order and the manifest grouping if any runtime relies on field ordering.
• Confirm LLM parameter changes are intentional and supported by deployment limits.
• Check package name change for any integration or CI references.

Possibly related PRs

• feat(tasks): update default LLM model #51 — modifies model configuration in tasks/translator/task.oo.yaml (related LLM parameter changes).
• Dev #52 — touches translated_file declarations and related task metadata (related input/grouping edits).

Pre-merge checks

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	No description was provided by the author, making it impossible to assess relevance to the changeset.	Add a pull request description explaining the changes, including rationale for the package rename, input reorganization, and model parameter adjustments.

✅ Passed checks (1 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title follows the required format with type and subject, and accurately summarizes the three main changes: package rename, input reordering, and model defaults update.

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9951d05 and cd94cee.

📒 Files selected for processing (1)

• package.oo.yaml (1 hunks)

🔇 Additional comments (1)

package.oo.yaml (1)

10-10: Package name now matches repository URL convention.

The change from books_translator to books-translator aligns the package name with the repository URL, improving naming consistency across the codebase. All related references have been updated, including paths in tasks/translator/__init__.py and locale strings in oo-locales/en.json.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 2d453eb and 9951d05.

📒 Files selected for processing (2)

• tasks/translator/__init__.py (1 hunks)
• tasks/translator/task.oo.yaml (3 hunks)

🔇 Additional comments (3)

tasks/translator/__init__.py (1)

16-16: LGTM: Field reordering is safe.

The reordering of translated_file within the Inputs TypedDict is a safe refactor. TypedDict field order doesn't affect runtime behavior in Python, and the field type remains correctly specified as str | None.

tasks/translator/task.oo.yaml (2)

47-47: LGTM: Description formatting updated.

The quote style change in the description is a minor formatting improvement.

---

113-114: Confirm model parameter changes have been applied; verify impact is documented and tested.

The LLM model parameters in the configuration have been updated to top_p: 1 and max_tokens: 8100. The old values (0.5 and 4096 respectively) are not present in the codebase.

However, no documentation or tests were found that validate the impact of these changes on translation quality, consistency, and API costs. Ensure that:

• The new parameter values have been tested and align with project requirements
• Changes are documented with rationale and any observed impact
• API cost implications have been evaluated

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

Fix typo in filter name.

Line 67 contains a typo: "EPUE" should be "EPUB". This typo could cause the file filter to display incorrectly in the UI.

Apply this diff to fix the typo:

-          - name: EPUE
+          - name: EPUB

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

	- group: Translate
	collapsed: true
	- handle: translated_file
	description: '%the-saved-path-of-the-translated-epub-file-if-not-filled-in-spec%'
	json_schema:
	type: string
	ui:widget: save
	ui:options:
	excludeAll: true
	filters:
	- name: EPUE
	extensions:
	- epub
	value:
	nullable: true
	- group: Translate
	collapsed: true
	- handle: translated_file
	description: '%the-saved-path-of-the-translated-epub-file-if-not-filled-in-spec%'
	json_schema:
	type: string
	ui:widget: save
	ui:options:
	excludeAll: true
	filters:
	- name: EPUB
	extensions:
	- epub
	value:
	nullable: true

🤖 Prompt for AI Agents

In tasks/translator/task.oo.yaml around lines 57 to 71, the file filter name
"EPUE" is a typo and should be "EPUB"; update the filters -> - name: EPUE to -
name: EPUB so the UI file filter displays correctly (keep the extensions list
unchanged).