🧹 remove unused sys import in generate_docs_export.py#2604
Conversation
Co-authored-by: SatoryKono <13055362+SatoryKono@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
Caution Review failedAn error occurred during the review process. Please try again later. 📝 WalkthroughWalkthroughMixed repo updates: documentation and agent paths moved from Changes
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Co-authored-by: SatoryKono <13055362+SatoryKono@users.noreply.github.com>
github-actions
bot
added
documentation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b2e38c474f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
scripts/ops/setup_skills.sh
Outdated
| REPO_ROOT="$(cd "$SCRIPT_DIR/../.." && pwd)" | ||
|
|
||
| SOURCE_ROOT="$REPO_ROOT/.codex/skills" | ||
| SOURCE_ROOT="$REPO_ROOT/.claude/skills" |
There was a problem hiding this comment.
Pointing SOURCE_ROOT at .claude/skills makes setup_skills.sh stop installing the repository’s Codex runtime skills (for example py-review-orchestrator and py-test-swarm, which live under .codex/skills and are referenced throughout the project docs/tooling). In practice, running this setup now populates $CODEX_HOME/skills with a different skill set and drops the BioETL Codex workflows, so downstream local agent workflows fail to load expected skills.
Useful? React with 👍 / 👎.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 4
Caution
Some comments are outside the diff and can’t be posted inline due to platform limitations.
⚠️ Outside diff range comments (4)
scripts/docs/check_doc_drift.py (2)
12-12:⚠️ Potential issue | 🟡 MinorStale docstring reference to
.codex/agents/.The docstring on line 12 still references
.codex/agents/as the canonical source, but the actualRUNTIME_MIRROR_RULESnow use.claude/agents/.📝 Proposed fix
- 6. Active runtime docs mirrors drift from canonical `.codex/agents/` sources + 6. Active runtime docs mirrors drift from canonical `.claude/agents/` sources🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@scripts/docs/check_doc_drift.py` at line 12, Update the stale docstring reference that mentions ".codex/agents/" to the current canonical path ".claude/agents/" so it matches the actual RUNTIME_MIRROR_RULES constant; locate the docstring near RUNTIME_MIRROR_RULES in check_doc_drift.py and replace the old `.codex/agents/` text with `.claude/agents/`.
655-659:⚠️ Potential issue | 🟡 MinorStale help text reference to
.codex docs.The
--runtime-mirrorshelp text still references "canonical .codex docs" but should reference.claudedocs.📝 Proposed fix
parser.add_argument( "--runtime-mirrors", action="store_true", - help="Check published runtime docs mirrors against canonical .codex docs", + help="Check published runtime docs mirrors against canonical .claude docs", )🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@scripts/docs/check_doc_drift.py` around lines 655 - 659, Update the help text for the parser argument defined in parser.add_argument("--runtime-mirrors", ...) to replace the stale reference "canonical .codex docs" with "canonical .claude docs"; modify the help string in the --runtime-mirrors argument so it reads something like "Check published runtime docs mirrors against canonical .claude docs"..claude/agents/py-doc-bot.md (1)
118-118:⚠️ Potential issue | 🟡 MinorStale ADR range in directory structure comment.
Line 118 still shows
ADR-001 through ADR-040but line 45 was updated to50 ADR (ADR-001..ADR-050). This creates internal inconsistency within the document.📝 Proposed fix
-| +-- decisions/ # ADRs (ADR-001 through ADR-040) +| +-- decisions/ # ADRs (ADR-001 through ADR-050)🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/agents/py-doc-bot.md at line 118, Update the stale ADR range string in the directory structure comment: find the line containing the text 'decisions/ # ADRs (ADR-001 through ADR-040)' and change it to match the updated header '50 ADR (ADR-001..ADR-050)' used earlier (or to 'ADRs (ADR-001..ADR-050)' for consistency). Ensure the comment text referencing ADR ranges (the literal 'ADR-001 through ADR-040') is replaced with the matching 'ADR-001..ADR-050' form so the document is internally consistent..claude/agents/py-config-bot.md (1)
88-110:⚠️ Potential issue | 🟠 MajorHierarchy diagram shows outdated directory structure that contradicts actual templates and codebase.
The hierarchy diagram still uses old directory names:
configs/pipelines/instead ofconfigs/entities/configs/dq/entities/instead ofconfigs/quality/configs/filter/entities/instead ofconfigs/filters/These paths don't exist in the actual codebase and contradict the template paths documented later in the file (lines 121, 156, 179). Update the hierarchy diagram to match the actual structure.
📝 Proposed fix to align hierarchy with templates
configs/ -├── pipelines/ +├── entities/ │ ├── _defaults.yaml # Глобальные дефолты │ ├── {provider}/ │ │ └── {entity}.yaml # Pipeline config │ └── composite/ │ └── {name}.yaml # Composite pipeline config -├── dq/ +├── quality/ │ ├── _defaults.yaml # DQ глобальные дефолты │ ├── providers/ │ │ └── {provider}.yaml # DQ дефолты провайдера -│ └── entities/ -│ └── {provider}/ -│ └── {entity}.yaml # DQ правила entity -├── filter/ +│ └── {provider}/ +│ └── {entity}.yaml # DQ правила entity +├── filters/ │ ├── _defaults.yaml # Filter глобальные дефолты -│ └── entities/ -│ └── {provider}/ -│ └── {entity}.yaml # Filter правила entity +│ └── {provider}/ +│ └── {entity}.yaml # Filter правила entity └── sources/ └── {provider}.yaml # API source config🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.claude/agents/py-config-bot.md around lines 88 - 110, The hierarchy diagram at the top must be updated to match the actual template names: replace configs/pipelines/ with configs/entities/, replace configs/dq/entities/ with configs/quality/, and replace configs/filter/entities/ with configs/filters/ so the diagram aligns with the later template references (lines referencing templates for entities, quality, and filters); update the listed examples (e.g., {provider}/{entity}.yaml and directories under dq/filter) to use the corrected directories (configs/entities/, configs/quality/, configs/filters/) so the diagram and templates are consistent.
🧹 Nitpick comments (3)
.github/workflows/duplication-complexity.yml (1)
205-207: Usepython -m pipwith pinned version for consistency and stability.Line 206 installs an unpinned dependency, which can cause version drift. The workflow already uses
python -m pipwith pinned versions elsewhere (e.g., line 42, 225). Align this install with that pattern by usingpython -m pip install "pyyaml==6.0.3"(version fromuv.lock).🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In @.github/workflows/duplication-complexity.yml around lines 205 - 207, In the "Install dependencies" workflow step named "Install dependencies" replace the unpinned pip invocation with a pinned, consistent installer invocation: use python -m pip to install pyyaml pinned to version 6.0.3 (the version from uv.lock) so the command aligns with other steps that use python -m pip and prevents version drift.src/bioetl/composition/runtime_builders/control_plane.py (1)
85-87: Usedataclasses.fields()instead ofhasattr()for filteringreplace()kwargs.Line 86 relies on
hasattr(ctx, k)to filter updates before passing toreplace(), butreplace()only accepts actual dataclass fields. While all current update keys are valid fields, usinghasattr()is implicit and fragile—it doesn't guarantee the attribute is a real dataclass field. With the dataclass usingslots=True, this is particularly important for clarity. Usefields(ctx)to explicitly filter by field names.♻️ Proposed change
-from dataclasses import is_dataclass, replace +from dataclasses import fields, is_dataclass, replace @@ if is_dataclass(ctx): - filtered_updates = { - k: v for k, v in updates.items() if hasattr(ctx, k) or k == "manifest_id" - } + field_names = {f.name for f in fields(ctx)} + filtered_updates = {k: v for k, v in updates.items() if k in field_names} return cast( "PipelineRunContext", replace(cast("DataclassInstance", ctx), **filtered_updates), )🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/bioetl/composition/runtime_builders/control_plane.py` around lines 85 - 87, The current filtering uses hasattr(ctx, k) before calling replace(), which is fragile and may include non-dataclass attributes; instead import dataclasses.fields and build a set of actual dataclass field names from fields(ctx) then filter updates with k in that set or k == "manifest_id" before calling replace() (update the code that creates filtered_updates and ensure fields(ctx) is used to compute permitted field names).configs/quality/scripts_inventory_manifest.json (1)
3944-3961: Add non-README coverage for the generic GitHub issue helper.Both references here come from
scripts/ops/README.md, butscripts/ops/update_github_issue.shis an active, generic mutator for issue title/body/comment/state. A lightweight architecture test for--helpand dry-run defaults would give future regenerations atestsreference here and make drift detection more useful than doc-only coverage.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@configs/quality/scripts_inventory_manifest.json` around lines 3944 - 3961, Add a lightweight test file that provides non-README coverage for the generic mutator scripts/ops/update_github_issue.sh: create a test (e.g., tests/scripts/test_update_github_issue.sh or a bats test) that invokes scripts/ops/update_github_issue.sh --help and asserts it exits 0 and prints expected usage text, and also invokes a dry-run mode (or equivalent default no-op flags the script supports) verifying it exits 0, emits the dry-run/no-op message and does not perform network calls; reference the script name scripts/ops/update_github_issue.sh and the flags --help and the script's dry-run option when writing assertions so future regenerations add this test as a non-README reference.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In @.claude/agents/py-plan-bot.md:
- Line 35: The providers string contains duplicate entries ("SemanticScholar"
and "OpenAlex"); update the provider list to include each provider only once by
removing the repeated tokens so it becomes e.g. "ChEMBL, PubChem, UniProt,
PubMed, CrossRef, OpenAlex, SemanticScholar" and ensure items remain
comma-separated and trimmed.
- Around line 207-212: The tool name OpenAlex:query_open_targets_graphql
violates the OpenAlex naming convention by embedding "open_targets"; rename it
to a consistent OpenAlex identifier (e.g., OpenAlex:query_entities_graphql) or
explicitly document that this tool queries Open Targets via OpenAlex; update any
references to OpenAlex:query_open_targets_graphql throughout the document and
adjust its description to reflect whether it targets OpenAlex entities or
proxies Open Targets data to avoid ambiguity with other tools like
OpenAlex:search_entities, bioRxiv:search_preprints, and PubMed:search_articles.
In @.github/workflows/type-checking.yml:
- Around line 39-41: The workflow step named "Install reports dependencies" is
installing lxml ad hoc; either remove that step entirely if lxml is unused, or
add lxml as a tracked dependency in pyproject.toml (e.g., under [project.dev] or
[project.optional-dependencies] with a pinned version) and update the workflow
to install dev dependencies from your lockfile or equivalent rather than running
`pip install lxml` directly; locate the "Install reports dependencies" job in
the type-checking workflow and make one of these two changes to eliminate the
untracked floating dependency.
In `@configs/quality/scripts_inventory_manifest.json`:
- Around line 3783-3795: Update the README entry in scripts/ops/README.md (the
table row referencing `setup-agents`/`scripts/ops/setup_agents.sh`) to
explicitly state the source directory and destination, e.g., change "Sync
project Codex agents into `CODEX_HOME`" to "Sync agents from `.claude/agents/`
into local `CODEX_HOME`" or similar wording like "Sync repository agents to
Codex runtime home" so it clearly reflects that setup_agents.sh copies agents
from `$REPO_ROOT/.claude/agents/` into the local `CODEX_HOME` (typically
`~/.codex/agents`).
---
Outside diff comments:
In @.claude/agents/py-config-bot.md:
- Around line 88-110: The hierarchy diagram at the top must be updated to match
the actual template names: replace configs/pipelines/ with configs/entities/,
replace configs/dq/entities/ with configs/quality/, and replace
configs/filter/entities/ with configs/filters/ so the diagram aligns with the
later template references (lines referencing templates for entities, quality,
and filters); update the listed examples (e.g., {provider}/{entity}.yaml and
directories under dq/filter) to use the corrected directories
(configs/entities/, configs/quality/, configs/filters/) so the diagram and
templates are consistent.
In @.claude/agents/py-doc-bot.md:
- Line 118: Update the stale ADR range string in the directory structure
comment: find the line containing the text 'decisions/ # ADRs
(ADR-001 through ADR-040)' and change it to match the updated header '50 ADR
(ADR-001..ADR-050)' used earlier (or to 'ADRs (ADR-001..ADR-050)' for
consistency). Ensure the comment text referencing ADR ranges (the literal
'ADR-001 through ADR-040') is replaced with the matching 'ADR-001..ADR-050' form
so the document is internally consistent.
In `@scripts/docs/check_doc_drift.py`:
- Line 12: Update the stale docstring reference that mentions ".codex/agents/"
to the current canonical path ".claude/agents/" so it matches the actual
RUNTIME_MIRROR_RULES constant; locate the docstring near RUNTIME_MIRROR_RULES in
check_doc_drift.py and replace the old `.codex/agents/` text with
`.claude/agents/`.
- Around line 655-659: Update the help text for the parser argument defined in
parser.add_argument("--runtime-mirrors", ...) to replace the stale reference
"canonical .codex docs" with "canonical .claude docs"; modify the help string in
the --runtime-mirrors argument so it reads something like "Check published
runtime docs mirrors against canonical .claude docs".
---
Nitpick comments:
In @.github/workflows/duplication-complexity.yml:
- Around line 205-207: In the "Install dependencies" workflow step named
"Install dependencies" replace the unpinned pip invocation with a pinned,
consistent installer invocation: use python -m pip to install pyyaml pinned to
version 6.0.3 (the version from uv.lock) so the command aligns with other steps
that use python -m pip and prevents version drift.
In `@configs/quality/scripts_inventory_manifest.json`:
- Around line 3944-3961: Add a lightweight test file that provides non-README
coverage for the generic mutator scripts/ops/update_github_issue.sh: create a
test (e.g., tests/scripts/test_update_github_issue.sh or a bats test) that
invokes scripts/ops/update_github_issue.sh --help and asserts it exits 0 and
prints expected usage text, and also invokes a dry-run mode (or equivalent
default no-op flags the script supports) verifying it exits 0, emits the
dry-run/no-op message and does not perform network calls; reference the script
name scripts/ops/update_github_issue.sh and the flags --help and the script's
dry-run option when writing assertions so future regenerations add this test as
a non-README reference.
In `@src/bioetl/composition/runtime_builders/control_plane.py`:
- Around line 85-87: The current filtering uses hasattr(ctx, k) before calling
replace(), which is fragile and may include non-dataclass attributes; instead
import dataclasses.fields and build a set of actual dataclass field names from
fields(ctx) then filter updates with k in that set or k == "manifest_id" before
calling replace() (update the code that creates filtered_updates and ensure
fields(ctx) is used to compute permitted field names).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 50a1c134-17c2-4758-95e2-89e32fc7ea00
📒 Files selected for processing (24)
.claude/agents/ORCHESTRATION.md.claude/agents/py-audit-bot.md.claude/agents/py-config-bot.md.claude/agents/py-doc-bot.md.claude/agents/py-plan-bot.md.github/workflows/duplication-complexity.yml.github/workflows/security.yml.github/workflows/type-checking.ymlconfigs/_schema/pipeline.jsonconfigs/quality/scripts_inventory_manifest.jsondocs/reports/evidence/project-package-topology/04-decisions/DECISIONS.yamldocs/reports/evidence/residual-test-ci-debt/06-backlog/BACKLOG-residual-test-ci-debt-implementation-2026-04-01.mddocs/reports/evidence/technical-debt/complexity-hotspots/SUMMARY.mdreports/quality/hotspot-duplication-baseline.jsonreports/quality/vcr-metadata-catalog.jsonscripts/docs/check_doc_drift.pyscripts/ops/setup_agents.shscripts/ops/setup_skills.shsrc/bioetl/composition/runtime_builders/control_plane.pysrc/bioetl/domain/lineage/__init__.pysrc/bioetl/infrastructure/schemas/pipeline_contract_policy.pysrc/bioetl/interfaces/cli/commands/domains/maintenance/plan.pytests/architecture/test_config_topology_docs_drift.pytests/architecture/test_ops_ai_setup_scripts.py
💤 Files with no reviewable changes (1)
- tests/architecture/test_ops_ai_setup_scripts.py
✅ Files skipped from review due to trivial changes (5)
- reports/quality/hotspot-duplication-baseline.json
- .claude/agents/ORCHESTRATION.md
- .claude/agents/py-audit-bot.md
- reports/quality/vcr-metadata-catalog.json
- src/bioetl/domain/lineage/init.py
.claude/agents/py-plan-bot.md
Outdated
| ### OpenAlex — валидация планов по таргетам | ||
|
|
||
| | Сценарий | Инструмент | Параметры | Результат | | ||
| |----------|------------|-----------|-----------| | ||
| | Проверка target existence | `Open Targets:search_entities` | `query_strings=["BRCA1"]` | ID resolution | | ||
| | Оценка data volume | `Open Targets:query_open_targets_graphql` | Query с counts | Capacity planning | | ||
| | Проверка target existence | `OpenAlex:search_entities` | `query_strings=["BRCA1"]` | ID resolution | | ||
| | Оценка data volume | `OpenAlex:query_open_targets_graphql` | Query с counts | Capacity planning | |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Search for OpenAlex MCP tool definitions or usages to verify correct tool names
rg -n --type=py -C3 'query_open_targets_graphql|search_entities' src/
rg -n --type=yaml 'query_open_targets_graphql|search_entities' configs/Repository: SatoryKono/BioactivityDataAcquisition
Length of output: 63
🏁 Script executed:
# Check if .claude/agents/ directory exists and understand its purpose
find . -name "py-plan-bot.md" -type f | head -5Repository: SatoryKono/BioactivityDataAcquisition
Length of output: 162
🏁 Script executed:
# Check the repository structure
git ls-files | head -30Repository: SatoryKono/BioactivityDataAcquisition
Length of output: 1121
🏁 Script executed:
# Search for any references to OpenAlex or MCP tool definitions
rg -n "OpenAlex|query_open_targets|search_entities" --type=md --type=py --type=yamlRepository: SatoryKono/BioactivityDataAcquisition
Length of output: 50394
🏁 Script executed:
# Check for MCP tool definitions or configurations
rg -n "MCP|mcp_tool|tool.*name" --type=py --type=yaml --type=json | head -30Repository: SatoryKono/BioactivityDataAcquisition
Length of output: 5269
🏁 Script executed:
cat -n .claude/agents/py-plan-bot.md | sed -n '200,220p'Repository: SatoryKono/BioactivityDataAcquisition
Length of output: 1258
🏁 Script executed:
# Check the full section around lines 207-212 to see context
cat -n .claude/agents/py-plan-bot.md | sed -n '190,230p'Repository: SatoryKono/BioactivityDataAcquisition
Length of output: 1838
🏁 Script executed:
# Look for other MCP tool definitions in the same document to understand naming patterns
rg -n ":\w+_" .claude/agents/py-plan-bot.md | head -20Repository: SatoryKono/BioactivityDataAcquisition
Length of output: 617
Update tool name to follow consistent OpenAlex naming pattern or clarify the intended behavior.
The tool name OpenAlex:query_open_targets_graphql contains a mismatched reference to "open_targets" within an OpenAlex section. Compared to other tools in the document (bioRxiv:search_preprints, OpenAlex:search_entities, PubMed:search_articles), this tool breaks the naming convention by embedding a nested provider reference. Either rename it to a generic identifier like OpenAlex:query_entities_graphql, or document if this tool is intentionally designed to query Open Targets data through OpenAlex.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In @.claude/agents/py-plan-bot.md around lines 207 - 212, The tool name
OpenAlex:query_open_targets_graphql violates the OpenAlex naming convention by
embedding "open_targets"; rename it to a consistent OpenAlex identifier (e.g.,
OpenAlex:query_entities_graphql) or explicitly document that this tool queries
Open Targets via OpenAlex; update any references to
OpenAlex:query_open_targets_graphql throughout the document and adjust its
description to reflect whether it targets OpenAlex entities or proxies Open
Targets data to avoid ambiguity with other tools like OpenAlex:search_entities,
bioRxiv:search_preprints, and PubMed:search_articles.
| "reference_count": 3, | ||
| "references": [ | ||
| { | ||
| "path": "Makefile", | ||
| "line": 89, | ||
| "source_group": "build", | ||
| "text": "@bash scripts/ops/setup_agents.sh" | ||
| }, | ||
| { | ||
| "path": "makefile", | ||
| "line": 89, | ||
| "source_group": "build", | ||
| "text": "@bash scripts/ops/setup_agents.sh" | ||
| }, | ||
| { | ||
| "path": "scripts/ops/README.md", | ||
| "line": 19, | ||
| "line": 26, | ||
| "source_group": "scripts", | ||
| "text": "| `setup-agents` | `scripts/ops/setup_agents.sh` | Sync project Codex agents into `CODEX_HOME` |" |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
rg -n -C2 'CODEX_HOME|CLAUDE|\.codex|\.claude' \
AGENTS.md \
scripts/ops/README.md \
scripts/ops/setup_agents.sh \
docs/00-project/ai \
docs/00-project/index.md || trueRepository: SatoryKono/BioactivityDataAcquisition
Length of output: 50395
Update README description to clarify agent source directory.
Lines 3793–3795 preserve the README text "Sync project Codex agents into CODEX_HOME", which is imprecise. The script actually syncs agents from $REPO_ROOT/.claude/agents/ into the local CODEX_HOME (typically ~/.codex/agents). Calling them "Codex agents" is misleading when they originate from the .claude/ directory.
Update scripts/ops/README.md line 26 to something like: "Sync agents from .claude/agents/ into local CODEX_HOME" or "Sync repository agents to Codex runtime home".
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@configs/quality/scripts_inventory_manifest.json` around lines 3783 - 3795,
Update the README entry in scripts/ops/README.md (the table row referencing
`setup-agents`/`scripts/ops/setup_agents.sh`) to explicitly state the source
directory and destination, e.g., change "Sync project Codex agents into
`CODEX_HOME`" to "Sync agents from `.claude/agents/` into local `CODEX_HOME`" or
similar wording like "Sync repository agents to Codex runtime home" so it
clearly reflects that setup_agents.sh copies agents from
`$REPO_ROOT/.claude/agents/` into the local `CODEX_HOME` (typically
`~/.codex/agents`).
Co-authored-by: SatoryKono <13055362+SatoryKono@users.noreply.github.com>
- Remove unused 'sys' import in src/tools/generate_docs_export.py - Fix unsorted import block in src/bioetl/domain/lineage/__init__.py - Ensure lint compliance (I001) for affected files Co-authored-by: SatoryKono <13055362+SatoryKono@users.noreply.github.com>
SatoryKono
deleted the
cleanup-unused-sys-import-generate-docs-export-15514756899629086213
branch
1 participant
🎯 What: Removed the unused
sysimport fromsrc/tools/generate_docs_export.py.💡 Why: This improves maintainability and readability by cleaning up dead code. The script uses
raise SystemExit(main())for its exit status, makingsysredundant.✅ Verification: Ran
uv run python src/tools/generate_docs_export.py --helpto ensure functionality is preserved. Validated withuv run ruff check src/tools/generate_docs_export.pyanduv run ruff format src/tools/generate_docs_export.py.✨ Result: A cleaner, more maintainable script with no functional changes.
PR created automatically by Jules for task 15514756899629086213 started by @SatoryKono
Summary by CodeRabbit