fix: convert remaining rst docstrings

[AngeloDanducci]

Pull Request

Issue

Fixes #1172

Description

Converts remaining RST docstrings

Testing

• Tests added to the respective file if code was changed
• New code has 100% coverage if code was added
• Ensure existing tests and github automation passes (a maintainer will kick off the github automation when the rest of the PR is populated)

Attribution

• AI coding assistants used

---

Adding a new component, requirement, sampling strategy, or tool?

If your PR adds or modifies one of the types below, check the matching box. A checklist of type-specific review items will be posted as a comment.

• Component
• Requirement
• Sampling Strategy
• Tool

NOTE: Please ensure you have an issue that has been acknowledged by a core contributor and routed you to open a pull request against this repository. Otherwise, please open an issue before continuing with this pull request.

[planetf1]

A few gaps in the RST conversion — details inline.

[planetf1]

Thanks for the follow-up commit — the issues I flagged in the first round are all resolved: the Example: and Deprecated: fences in interpreter.py are in place, the CONTRIBUTING.md fence nesting is correct (4-backtick outer, 3-backtick inner), and the documents type annotation in intrinsic/core.py is now present. Approving with a few small suggestions below.

[planetf1]

Nice catch making the fences RST-aware. One gap: the optional third backtick means we now also match python `` blocks, but all the existing tests in test_reqlib_python.py use triple-backtick Markdown — which the old regex already matched. So the new double-backtick path has no coverage. Worth adding a test case that feeds in something like:

``python
def f():
    return 1
``

...and asserts the block is extracted. That locks in the behaviour this change is actually for.

[planetf1]

Cosmetic only (ruff doesn't flag docstring prose), but this line runs long with the fully-qualified type inline. Breaking after the type keeps it aligned with the other args.

Suggested change

	documents (collections.abc.Iterable[str | Document]): Documents used to generate `response`. Each element may be a
	documents (collections.abc.Iterable[str | Document]): Documents used to
	generate `response`. Each element may be a

[planetf1]

Not a regression — the old :returns: was just as vague — but since we're touching this, the signature gives us the element types and the conversion was a chance to make the Returns line precise.

Suggested change

	Returns:
	tuple[SystemMessage | None, list]: The system message if present
	(else `None`), and the remaining non-system messages.

[planetf1]

Tiny style nit while we're tidying these — Google style puts Note: on its own line with the body indented under it, matching how Args:/Raises: render. Inline works but is slightly inconsistent.

Suggested change

	Note: Prefer `log_context` as the primary API — it guarantees cleanup
	(including restoring outer values on same-key nesting) even on exceptions.
	Note:
	Prefer `log_context` as the primary API — it guarantees cleanup
	(including restoring outer values on same-key nesting) even on exceptions.