Repository navigation
isort #178
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
isort #178
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -20,13 +20,10 @@ | |||||||||||||||||||||||
| from pathlib import Path | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| from quest_runtime.artifacts import expected_artifacts_for_role | ||||||||||||||||||||||||
| from quest_runtime.claude_runner import ( | ||||||||||||||||||||||||
| DEFAULT_BG_RUNNER_SCRIPT, | ||||||||||||||||||||||||
| DEFAULT_BRIDGE_SCRIPT, | ||||||||||||||||||||||||
| resolve_claude_transport, | ||||||||||||||||||||||||
| resolve_path, | ||||||||||||||||||||||||
| run_claude_role, | ||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||
| from quest_runtime.claude_runner import (DEFAULT_BG_RUNNER_SCRIPT, | ||||||||||||||||||||||||
| DEFAULT_BRIDGE_SCRIPT, | ||||||||||||||||||||||||
| resolve_claude_transport, | ||||||||||||||||||||||||
| resolve_path, run_claude_role) | ||||||||||||||||||||||||
|
Comment on lines
+23
to
+26
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: This isort default-profile format will fail the repo's black check: CI runs Prompt for AI agents
Suggested change
|
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| _TELEMETRY_ENV = "QUEST_RUNNER_TELEMETRY_LOG" | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -13,12 +13,13 @@ | |||||||||||||||||||||||||||
| from datetime import date, datetime, timezone | ||||||||||||||||||||||||||||
| from pathlib import Path | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| from quest_celebrate.quest_data import ( | ||||||||||||||||||||||||||||
| QUALITY_TIERS, | ||||||||||||||||||||||||||||
| extract_celebration_data_from_journal as _extract_celebration_data, | ||||||||||||||||||||||||||||
| extract_metadata_value as _extract_metadata, | ||||||||||||||||||||||||||||
| friendly_model_name as _friendly_model_name, | ||||||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||||||
| from quest_celebrate.quest_data import QUALITY_TIERS | ||||||||||||||||||||||||||||
| from quest_celebrate.quest_data import \ | ||||||||||||||||||||||||||||
| extract_celebration_data_from_journal as _extract_celebration_data | ||||||||||||||||||||||||||||
| from quest_celebrate.quest_data import \ | ||||||||||||||||||||||||||||
| extract_metadata_value as _extract_metadata | ||||||||||||||||||||||||||||
| from quest_celebrate.quest_data import \ | ||||||||||||||||||||||||||||
| friendly_model_name as _friendly_model_name | ||||||||||||||||||||||||||||
|
Comment on lines
+16
to
+22
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: This change breaks the repo's enforced formatting gate. Prompt for AI agents
Suggested change
|
||||||||||||||||||||||||||||
| from quest_runtime.quest_ids import parse_quest_id | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| from .models import ActiveQuest, DashboardData, JournalEntry | ||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -7,13 +7,10 @@ | |||||||||||||||||||||||
| import json | ||||||||||||||||||||||||
| import sys | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| from quest_runtime.plan_iterations import ( | ||||||||||||||||||||||||
| PlanIterationError, | ||||||||||||||||||||||||
| cleanup_current, | ||||||||||||||||||||||||
| publish_refinement, | ||||||||||||||||||||||||
| snapshot_plan_iteration, | ||||||||||||||||||||||||
| verify_refinement, | ||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||
| from quest_runtime.plan_iterations import (PlanIterationError, cleanup_current, | ||||||||||||||||||||||||
| publish_refinement, | ||||||||||||||||||||||||
| snapshot_plan_iteration, | ||||||||||||||||||||||||
| verify_refinement) | ||||||||||||||||||||||||
|
Comment on lines
+10
to
+13
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: This isort hanging-grid import is not black-compliant and will fail CI's Prompt for AI agents
Suggested change
|
||||||||||||||||||||||||
| from quest_runtime.state import StateError | ||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
|
|
||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -10,14 +10,9 @@ | |||||||||||||||||||||||||
| from pathlib import Path | ||||||||||||||||||||||||||
| from typing import Any | ||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| from quest_runtime.pr_shepherd import ( | ||||||||||||||||||||||||||
| ADDRESSED_MARKER, | ||||||||||||||||||||||||||
| FOLLOWUP_MARKER, | ||||||||||||||||||||||||||
| SUMMARY_MARKER, | ||||||||||||||||||||||||||
| append_marker, | ||||||||||||||||||||||||||
| compact_summary_body, | ||||||||||||||||||||||||||
| has_marker, | ||||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||||
| from quest_runtime.pr_shepherd import (ADDRESSED_MARKER, FOLLOWUP_MARKER, | ||||||||||||||||||||||||||
| SUMMARY_MARKER, append_marker, | ||||||||||||||||||||||||||
| compact_summary_body, has_marker) | ||||||||||||||||||||||||||
|
Comment on lines
+13
to
+15
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: This isort grid layout conflicts with the repo's enforced formatter: CI runs Prompt for AI agents
Suggested change
|
||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| PER_PAGE = 100 | ||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||
| Original file line number | Diff line number | Diff line change | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
|
|
@@ -8,25 +8,20 @@ | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from pathlib import Path | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from typing import Any | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from quest_runtime.pr_review_cycle import ( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| allowlist_path_from_context, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| build_fix_batches, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| classify_pr_operational_state, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| classify_pr_loop_stop, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| normalize_pr_review_intake, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| retag_backlog_at_cap, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| select_validation_steps, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from quest_runtime.review_intelligence import ( | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| append_deferred_findings, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| build_review_backlog, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| merge_and_dedupe, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| scan_deferred_backlog, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| utc_now_iso, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| validate_plan_phase_defaults, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| validate_review_backlog, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| validate_findings, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| ) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from quest_runtime.pr_review_cycle import (allowlist_path_from_context, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| build_fix_batches, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| classify_pr_loop_stop, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| classify_pr_operational_state, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| normalize_pr_review_intake, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| retag_backlog_at_cap, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| select_validation_steps) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| from quest_runtime.review_intelligence import (append_deferred_findings, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| build_review_backlog, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| merge_and_dedupe, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| scan_deferred_backlog, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| utc_now_iso, validate_findings, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| validate_plan_phase_defaults, | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
Comment on lines
+11
to
+23
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: This isort reformatting produces grid-style imports (first name on the opening-paren line, aligned continuation, no trailing comma) that conflict with black, the repo's enforced formatter: CI runs Prompt for AI agents
Suggested change
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| validate_review_backlog) | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| def _load_json(path: Path) -> Any: | ||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
P3: This isort correction isn't reproducible: the repo has no isort config (
pyproject.tomlhas no[tool.isort]and no.isort.cfgexists), and this output doesn't match what isort emits under its defaults or the black profile. The two-namequest_dataimport is split across lines at 76 chars, which a default isort run would collapse back onto one line (defaultline_length79);multi_line_output3 with trailing comma would also reformat thequest_completeimport. A laterisortpass will churn this change back. Either add an isort configuration (e.g. themulti_line_output = 4grid style this file now uses) topyproject.tomlso the output is reproducible and enforced, or drop the isort claim and keep the repo's existing grid convention.Prompt for AI agents