chore: trivial idiom cleanups (ruff PIE810, FURB167, B009)

[alexzhu0]

Summary

Three ruff-autofix rule groups bundled together. All 15 changes are semantics-preserving Python idiom cleanups.

Changes

Rule	Count	Before → After
PIE810	5	x.startswith("a") or x.startswith("b") → x.startswith(("a", "b"))
FURB167	5	re.I, re.M → re.IGNORECASE, re.MULTILINE
B009	5	getattr(obj, "id") → obj.id

PIE810 sites

• api/v1/prompts/routers/get_prompts_router.py (3×)
• memory/episodic/routing/document_router.py
• shared/files/utils/get_data_file_path.py

str.startswith accepts a tuple of prefixes natively; one call instead of two.

FURB167 sites

• knowledge/summarization/summarize_by_event.py
• knowledge/summarization/text_summary_parser.py (3×)
• memory/procedural/procedural_points_builder.py

The full names (re.IGNORECASE, re.MULTILINE) are the canonical form in Python's re docs; single-letter aliases are easy to misread.

B009 sites

• retrieval/episodic/bundle_search.py (3×)
• retrieval/utils/fine_grained_triplet_search.py
• retrieval/utils/procedural_bundle_search.py

getattr(obj, "id") with a constant attribute offers no safety over obj.id, and hides typos from IDE tooling / static analysis.

Verification

$ ruff check <touched files> --select B009,FURB167,PIE810
All checks passed!

$ python -m py_compile <touched files>   # clean

I affirm that all code in every commit of this pull request conforms to the terms of the M-flow Developer Certificate of Origin

[FlowElement-ai]

Thanks for this cleanup! Two small things hold this back from merging:

1. DCO: I added the affirmation to the PR body for you, the Verify DCO statement check should pass now.
2. Fork PR lightweight checks still fails because m_flow/ingestion/documents/classify_documents.py (the file your dict.fromkeys autofix touches) is unformatted under the repo-pinned ruff 0.13.1 in uv.lock. Newer ruff joins the 16-element image-extension list into one line (>120 chars), but 0.13.1 wants the list re-expanded.

To unblock, please run locally and push:

uv sync --group dev
uv run ruff format m_flow/ingestion/documents/classify_documents.py
git commit --amend --no-edit && git push --force-with-lease

Same idea applies to your other open PRs that 're hitting the same fork-lite check. After that the queue should go green and we can merge the batch. Thanks!

[FlowElement-ai]

Quick update: I just merged #142 which bumps the lockfile ruff from 0.13.1 → 0.15.10 to match the pyproject.toml ceiling. That was the version that disagreed with your autofix output on every recent fork PR.

Triggered a fork-lite rerun on this PR with the new lock — it still failed, which means your ruff is even newer than 0.15.10. The smallest unblock is one of:

# either downgrade locally to match the repo
uv tool install ruff@0.15.10
ruff format .

# or just re-run with whatever ruff comes from the project venv
uv run ruff format .

git add -u && git commit --amend --no-edit && git push --force-with-lease

After that the fork-lite check should go green and we can merge. Sorry for the back and forth on this one — the lockfile drift was the underlying cause.