Pythagora-io/gpt-pilot

Pattern Findings

358 matches across 15 categories. Click a row to expand file-level details.

Hyper-Verbose Identifiers155 hits · 161 pts

Severity	File	Line	Snippet
LOW	core/ui/api_server.py	511	async def _handle_current_task_status(self, message: Message, writer: asyncio.StreamWriter):
LOW	core/ui/api_server.py	600	async def _handle_start_another_task(self, message: Message, writer: asyncio.StreamWriter):
LOW	core/config/__init__.py	177	def from_provider_and_agent_configs(cls, provider: ProviderConfig, agent: AgentLLMConfig):
LOW	core/agents/mixins.py	130	async def get_relevant_files_parallel(
LOW	core/agents/convo.py	41	def _get_default_template_vars(self) -> dict:
LOW	core/agents/convo.py	55	def _serialize_prompt_context(context: dict) -> dict:
LOW	core/agents/tech_lead.py	92	def create_initial_project_epic(self):
LOW	core/agents/bug_hunter.py	82	async def get_bug_reproduction_instructions(self):
LOW	core/agents/bug_hunter.py	363	def generate_iteration_convo_so_far(self, omit_last_cycle=False):
LOW	core/agents/bug_hunter.py	400	def set_data_for_next_hunting_cycle(self, human_readable_instructions, new_status):
LOW	core/agents/problem_solver.py	49	async def generate_alternative_solutions(self):
LOW	core/agents/problem_solver.py	76	async def try_alternative_solutions(self) -> AgentResponse:
LOW	core/agents/problem_solver.py	107	async def ask_for_preferred_solution(self) -> Optional[tuple[int, str]]:
LOW	core/agents/frontend.py	33	def has_correct_num_of_backticks(response: str) -> bool:
LOW	core/agents/architect.py	237	async def check_system_dependencies(self, spec: Specification):
LOW	core/agents/orchestrator.py	345	def handle_parallel_responses(self, agent: BaseAgent, responses: List[AgentResponse]) -> AgentResponse:
LOW	core/agents/troubleshooter.py	348	def try_next_alternative_solution(self, user_feedback: str, user_feedback_qa: list[str]) -> AgentResponse:
LOW	core/agents/developer.py	121	async def breakdown_current_iteration(self) -> AgentResponse:
LOW	core/agents/spec_writer.py	63	async def initialize_spec_and_project(self) -> AgentResponse:
LOW	core/cli/helpers.py	412	def find_first_todo_task_index(tasks):
LOW	core/cli/helpers.py	849	async def list_projects_branches_states(db: SessionManager):
LOW	core/state/state_manager.py	106	async def list_projects_with_branches_states(self) -> list[Project]:
LOW	core/state/state_manager.py	116	async def get_branches_for_project_id(self, project_id: UUID) -> list[Branch]:
LOW	core/state/state_manager.py	119	async def get_project_state_for_redo_task(self, project_state: ProjectState) -> Optional[ProjectState]:
LOW	core/state/state_manager.py	137	async def get_project_state_for_convo_id(self, convo_id) -> Optional["ProjectState"]:
LOW	core/state/state_manager.py	140	async def get_task_conversation_project_states(
LOW	core/state/state_manager.py	151	async def get_project_states_in_between(
LOW	core/state/state_manager.py	679	def get_full_parent_project_root(self) -> str:
LOW	core/state/state_manager.py	786	async def get_modified_files_with_content(self) -> list[dict]:
LOW	core/state/state_manager.py	850	async def update_implemented_pages_and_apis(self):
LOW	core/state/state_manager.py	950	async def find_backend_implementation(self, endpoint_line: str) -> dict:
LOW	core/db/models/project_state.py	370	def flag_iterations_as_modified(self):
LOW	core/db/models/project_state.py	400	def flag_knowledge_base_as_modified(self):
LOW	core/db/models/project_state.py	711	async def get_project_states_in_between(
LOW	core/db/models/project_state.py	758	async def get_task_conversation_project_states(
LOW	core/db/models/chat_convo.py	38	async def get_project_state_for_convo_id(session: AsyncSession, convo_id) -> Optional["ProjectState"]:
LOW	core/db/models/project.py	103	async def get_branches_for_project_id(session: AsyncSession, project_id: UUID) -> list["Branch"]:
LOW	core/db/models/project.py	117	async def get_all_projects_with_branches_states(session: "AsyncSession") -> list["Project"]:
LOW	core/db/models/project.py	153	def get_folder_from_project_name(name: str):
LOW	…tes/tree/vite_react/client/src/components/ui/chart.tsx	321	function getPayloadConfigFromPayload(
LOW	…/vite_react_swagger/client/src/components/ui/chart.tsx	321	function getPayloadConfigFromPayload(
LOW	tests/proc/test_process_manager.py	14	async def test_local_process_start_terminate(tmp_path):
LOW	tests/proc/test_process_manager.py	58	async def test_process_manager_run_command_capture_stdout(tmp_path):
LOW	tests/proc/test_process_manager.py	75	async def test_process_manager_run_command_capture_stderr(tmp_path):
LOW	tests/proc/test_process_manager.py	92	async def test_process_manager_start_list_terminate(tmp_path):
LOW	tests/ui/test_console.py	57	async def test_ask_question_with_buttons(mock_PromptSession):
LOW	tests/ui/test_console.py	78	async def test_ask_question_interrupted(mock_PromptSession):
LOW	tests/ui/test_ipc_client.py	193	async def test_server_closes_connection():
LOW	tests/ui/test_ipc_client.py	227	async def test_ask_question_buttons():
LOW	tests/ui/test_ipc_client.py	256	async def test_ask_question_buttons_only_with_default():
LOW	tests/ui/test_ipc_client.py	286	async def test_handle_garbage_response():
LOW	tests/llm/test_parser.py	25	def test_multi_code_block_parser(input, expected):
LOW	tests/llm/test_parser.py	123	def test_parse_json_with_spec(input, expected):
LOW	tests/llm/test_parser.py	204	def test_optional_block_parser(input, expected):
LOW	tests/llm/test_openai.py	68	async def test_openai_stream_handler(mock_AsyncOpenAI, mock_state_manager):
LOW	tests/llm/test_openai.py	98	async def test_openai_parser_with_retries(mock_AsyncOpenAI, mock_state_manager):
LOW	tests/llm/test_openai.py	179	async def test_openai_error_handler_success(mock_AsyncOpenAI, mock_state_manager):
LOW	tests/llm/test_openai.py	226	async def test_openai_error_handler_failure(mock_AsyncOpenAI, mock_state_manager):
LOW	tests/llm/test_openai.py	271	def test_openai_rate_limit_parser(
LOW	tests/llm/test_prompt.py	19	def test_jinja_string_template():
95 more matches not shown…

Excessive Try-Catch Wrapping68 hits · 80 pts

Severity	File	Line	Snippet
LOW	core/ui/ipc_client.py	213	except Exception as err:
LOW	core/ui/ipc_client.py	541	except Exception:
LOW	core/ui/api_server.py	243	except Exception as e:
LOW	core/ui/api_server.py	353	except Exception as err:
LOW	core/ui/api_server.py	430	except Exception as err:
LOW	core/ui/api_server.py	445	except Exception as err:
LOW	core/ui/api_server.py	480	except Exception as err:
LOW	core/ui/api_server.py	507	except Exception as err:
LOW	core/ui/api_server.py	536	except Exception as err:
LOW	core/ui/api_server.py	596	except Exception as err:
LOW	core/ui/api_server.py	635	except Exception as e:
LOW	core/ui/api_server.py	657	except Exception as err:
LOW	core/ui/api_server.py	698	except Exception as err:
LOW	core/ui/api_server.py	731	except Exception as err:
LOW	core/ui/api_server.py	773	except Exception as err:
LOW	core/llm/parser.py	183	except Exception as err:
LOW	core/llm/base.py	318	except Exception:
LOW	core/llm/base.py	344	except Exception:
LOW	core/llm/base.py	357	except Exception as e:
LOW	core/llm/relace_client.py	56	except Exception as e:
LOW	core/agents/mixins.py	185	except Exception as e:
LOW	core/agents/git.py	64	except Exception as e:
LOW	core/agents/frontend.py	263	except Exception as e:
LOW	core/agents/frontend.py	270	except Exception as e:
LOW	core/agents/frontend.py	504	except Exception as e:
LOW	core/agents/frontend.py	510	except Exception as e:
LOW	core/agents/frontend.py	597	except Exception as e:
LOW	core/agents/wizard.py	55	except Exception as e:
LOW	core/agents/wizard.py	162	except Exception as e:
LOW	core/agents/wizard.py	167	except Exception as e:
LOW	core/agents/orchestrator.py	219	except Exception as e:
LOW	core/agents/orchestrator.py	249	except Exception as e:
LOW	core/agents/orchestrator.py	280	except Exception as e:
LOW	core/agents/orchestrator.py	303	except Exception as e:
LOW	core/agents/orchestrator.py	342	except Exception as e:
LOW	core/agents/code_monkey.py	131	except Exception:
LOW	core/utils/text.py	40	except Exception:
LOW	core/cli/main.py	98	except Exception as err:
LOW	core/cli/main.py	417	except Exception as err:
MEDIUM	core/cli/helpers.py	300	print(f"Error parsing config file {args.config}: {err}", file=sys.stderr)
MEDIUM	core/state/state_manager.py	418	def commit_with_retry(self):
LOW	core/state/state_manager.py	237	except Exception as e:
LOW	core/state/state_manager.py	242	except Exception as e:
LOW	core/state/state_manager.py	368	except Exception as e:
LOW	core/state/state_manager.py	429	except Exception as e:
LOW	core/state/state_manager.py	475	except Exception as e:
LOW	core/state/state_manager.py	509	except Exception as e:
LOW	core/state/state_manager.py	520	except Exception as e:
LOW	core/state/state_manager.py	984	except Exception as e:
MEDIUM	core/db/fix_migrations.py	29	print(f"Error updating database: {e}")
MEDIUM	core/db/fix_migrations.py	42	print(f"Error: Could not find alembic.ini at {alembic_ini}")
MEDIUM	core/db/fix_migrations.py	50	print("Error: This script only works with SQLite databases")
MEDIUM	core/db/fix_migrations.py	69	print("Error: No migration versions found")
LOW	core/db/fix_migrations.py	84	except Exception as e:
MEDIUM	core/db/fix_migrations.py	85	print(f"Error: {e}")
LOW	core/db/v0importer.py	43	except Exception as err: # noqa
MEDIUM	core/db/v0importer.py	40	def import_database(self):
LOW	core/db/models/project_state.py	704	except Exception as e:
LOW	core/templates/base.py	126	except Exception as err:
MEDIUM	core/templates/base.py	123	def install_hook_template(self) -> Any:
8 more matches not shown…

Unused Imports43 hits · 42 pts

Severity	File	Line
LOW	core/config/__init__.py	256
LOW	core/agents/convo.py	15
LOW	core/agents/response.py	7
LOW	core/agents/response.py	8
LOW	core/agents/base.py	8
LOW	core/state/state_manager.py	17
LOW	core/state/state_manager.py	42
LOW	core/db/models/user_input.py	14
LOW	core/db/models/user_input.py	14
LOW	core/db/models/project_state.py	17
LOW	core/db/models/project_state.py	17
LOW	core/db/models/project_state.py	17
LOW	core/db/models/chat_message.py	9
LOW	core/db/models/__init__.py	6
LOW	core/db/models/__init__.py	7
LOW	core/db/models/__init__.py	8
LOW	core/db/models/__init__.py	9
LOW	core/db/models/__init__.py	10
LOW	core/db/models/__init__.py	11
LOW	core/db/models/__init__.py	12
LOW	core/db/models/__init__.py	13
LOW	core/db/models/__init__.py	14
LOW	core/db/models/__init__.py	15
LOW	core/db/models/__init__.py	16
LOW	core/db/models/__init__.py	17
LOW	core/db/models/__init__.py	17
LOW	core/db/models/__init__.py	18
LOW	core/db/models/branch.py	12
LOW	core/db/models/branch.py	14
LOW	core/db/models/branch.py	14
LOW	core/db/models/branch.py	14
LOW	core/db/models/branch.py	14
LOW	core/db/models/file.py	6
LOW	core/db/models/file.py	12
LOW	core/db/models/file.py	12
LOW	core/db/models/exec_log.py	13
LOW	core/db/models/exec_log.py	13
LOW	core/db/models/llm_request.py	14
LOW	core/db/models/llm_request.py	15
LOW	core/db/models/llm_request.py	15
LOW	core/templates/render.py	1
LOW	core/templates/base.py	13
LOW	core/templates/base.py	14

Deep Nesting32 hits · 31 pts

Severity	File	Line
LOW	core/proc/process_manager.py	171
LOW	core/ui/api_server.py	540
LOW	core/llm/base.py	124
LOW	core/llm/base.py	416
LOW	core/config/env_importer.py	44
LOW	core/agents/tech_lead.py	146
LOW	core/agents/tech_lead.py	371
LOW	core/agents/bug_hunter.py	61
LOW	core/agents/bug_hunter.py	241
LOW	core/agents/frontend.py	44
LOW	core/agents/frontend.py	151
LOW	core/agents/frontend.py	356
LOW	core/agents/frontend.py	451
LOW	core/agents/wizard.py	30
LOW	core/agents/wizard.py	133
LOW	core/agents/orchestrator.py	370
LOW	core/agents/orchestrator.py	463
LOW	core/agents/orchestrator.py	554
LOW	core/agents/troubleshooter.py	244
LOW	core/agents/developer.py	121
LOW	core/cli/helpers.py	437
LOW	core/cli/helpers.py	481
LOW	core/cli/helpers.py	583
LOW	core/state/state_manager.py	172
LOW	core/state/state_manager.py	272
LOW	core/db/models/project_state.py	592
LOW	core/db/models/project_state.py	631
LOW	core/db/models/project_state.py	758
LOW	core/db/models/project_state.py	976
LOW	core/db/models/file.py	46
LOW	core/templates/render.py	71
LOW	core/log/__init__.py	30

Redundant / Tautological Comments14 hits · 22 pts

Severity	File	Line	Snippet
LOW	core/ui/api_server.py	202	# Check if convo exists for this convo id
LOW	core/ui/api_server.py	213	# Check if any ChatMessages exists with the convo id
LOW	core/agents/git.py	99	# Check if there are any changes to commit
LOW	core/agents/orchestrator.py	187	# Check if file exists
LOW	core/agents/orchestrator.py	196	# Check if script already exists
LOW	core/agents/orchestrator.py	269	# Check if favicon link already exists
LOW	core/agents/orchestrator.py	318	# Check if required configs already exist
LOW	core/agents/spec_writer.py	70	# Check if initial_prompt is provided in command line arguments
LOW	core/cli/main.py	131	# Check if initial_prompt is provided, if so, automatically select "node"
LOW	core/state/state_manager.py	859	# Check if pages or apis have changed
LOW	core/templates/tree/add_raw_tags.py	8	# Open the file and read the contents
LOW	core/templates/tree/add_raw_tags.py	12	# Check if the tags are already present
LOW	core/templates/tree/add_raw_tags.py	42	# Check if the directory path argument is provided
LOW	core/templates/tree/add_raw_tags.py	50	# Check if the provided directory exists

Dead Code10 hits · 20 pts

Severity	File	Line
MEDIUM	core/agents/external_docs.py	50
MEDIUM	core/agents/external_docs.py	56
MEDIUM	core/agents/external_docs.py	57
MEDIUM	core/agents/external_docs.py	59
MEDIUM	core/agents/external_docs.py	64
MEDIUM	core/agents/external_docs.py	65
MEDIUM	core/agents/external_docs.py	66
MEDIUM	core/agents/external_docs.py	67
MEDIUM	core/agents/external_docs.py	69
MEDIUM	core/agents/external_docs.py	70

Self-Referential Comments6 hits · 18 pts

Severity	File	Line	Snippet
MEDIUM	core/llm/parser.py	186	# Create a new model that includes the original model fields and the original text
MEDIUM	core/agents/wizard.py	113	# Create a new knowledge base instance for the project state
MEDIUM	core/db/models/project_state.py	411	# Create a new knowledge base instance with the current data
MEDIUM	tests/cli/test_cli.py	308	# Create a mock with a string value for the button attribute
MEDIUM	tests/cli/test_cli.py	341	# Create a mock response with a string value for the button attribute
MEDIUM	tests/state/test_state_manager.py	143	# Create an instance of StateManager with mocked UI

Cross-File Repetition3 hits · 15 pts

Severity	File	Snippet
HIGH	core/state/state_manager.py	get a file from the current project state, by the file path. :param path: the file path. :return: the file object, or no
HIGH	core/db/models/project_state.py	get a file from the current project state, by the file path. :param path: the file path. :return: the file object, or no
HIGH	core/db/models/project_state.py	get a file from the current project state, by the file path. :param path: the file path. :return: the file object, or no

Magic Placeholder Names2 hits · 10 pts

Severity	File	Line	Snippet
HIGH	example-config.json	16	"api_key": "your-api-key",
HIGH	example-config.json	23	"api_key": "your-api-key",

Over-Commented Block10 hits · 10 pts

Severity	File	Line	Snippet
LOW	core/llm/base.py	301	# Token limit exceeded (in original gpt-pilot handled as
LOW	core/agents/tech_lead.py	61	if len(self.current_state.epics) == 1:
LOW	core/agents/architect.py	141	"select_templates",
LOW	core/db/alembic.ini	1	# A generic, single database configuration.
LOW	core/db/alembic.ini	21	# string value is passed to ZoneInfo()
LOW	core/db/alembic.ini	41	# The path separator used here should be the separator specified by "version_path_separator" below.
LOW	core/db/alembic.ini	61	# are written from script.py.mako
LOW	core/telemetry/__init__.py	141	# End result of development:
LOW	tests/db/test_db.py	61	# this assertion would fail with an ORM exception, unless we add
LOW	cloud/entrypoint.sh	1	#!/bin/bash

Fake / Example Data9 hits · 10 pts

Severity	File	Line	Snippet
LOW	tests/llm/test_convo.py	88	convo.user("Hello, World!", "John Doe")
LOW	tests/llm/test_convo.py	92	"name": "John Doe",
LOW	tests/integration/llm/test_anthropic.py	113	large_convo = " ".join(["lorem ipsum dolor sit amet"] * 60000)
LOW	tests/integration/llm/test_anthropic.py	113	large_convo = " ".join(["lorem ipsum dolor sit amet"] * 60000)
LOW	tests/integration/llm/test_openai.py	116	large_convo = " ".join(["lorem ipsum dolor sit amet"] * 30000)
LOW	tests/integration/llm/test_openai.py	116	large_convo = " ".join(["lorem ipsum dolor sit amet"] * 30000)
LOW	tests/integration/llm/test_groq.py	118	large_convo = " ".join(["lorem ipsum dolor sit amet"] * 30000)
LOW	tests/integration/llm/test_groq.py	118	large_convo = " ".join(["lorem ipsum dolor sit amet"] * 30000)
LOW	tests/telemetry/test_telemetry.py	36	"user_contact": "user@example.com",

Decorative Section Separators2 hits · 6 pts

Severity	File	Line	Snippet
MEDIUM	core/agents/code_monkey.py	209	# ------------------------------
MEDIUM	core/agents/code_monkey.py	211	# ------------------------------

Cross-Language Confusion1 hit · 5 pts

Severity	File	Line	Snippet
HIGH	core/agents/git.py	129	"git diff --cached \|\| git diff",

Slop Phrases2 hits · 4 pts

Severity	File	Line	Snippet
MEDIUM	core/llm/base.py	366	"Here's how you can get Pythagora to ignore those extra files: ",
LOW	core/agents/troubleshooter.py	241	# Sometimes LLM can return a non-existent file, let's make sure to filter those out

AI Slop Vocabulary1 hit · 2 pts

Severity	File	Line	Snippet
LOW	core/ui/api_server.py	541	# if the extension sent "add", just add a new task in the tasks array before the first todo task

Score History

Severity Breakdown

Pattern Findings