facebook/sapling

Pattern Findings

The scanner identified 6375 distinct pattern matches across 28 syntactic categories. Each entry below represents a discrete location in the source code where the engine recorded a detector match to inspect in context. Expand any category row to inspect the individual file paths, line numbers, code snippets, and the lexical context (CODE, COMMENT, or STRING) in which each match was detected.

Reading the findings table: The Severity column indicates the configured finding weight (CRITICAL / HIGH / MEDIUM / LOW). The Context column identifies whether the match occurred inside executable code, an inline comment, or a string literal — legacy comment-context matches receive a ×1.5 score weight. The ⚡ bolt icon marks clustered matches: three or more patterns within a 10-line window, each receiving an additional ×1.5 density multiplier under the legacy scoring model.

Over-Commented Block1997 hits · 1970 pts

Severity	File	Line	Snippet	Context
LOW	make-client.py	1	#!/usr/bin/env python3	COMMENT
LOW	eden/Eden.project.toml	1	# When working on Eden in Nuclide, it is helpful to scope	COMMENT
LOW	eden/integration/thrift_test.py	121	self.repo.write_file("local_dir/file", "hola\n")	COMMENT
LOW	eden/integration/mmap_test.py	61	self.assertEqual(0, m[i])	COMMENT
LOW	eden/integration/projfs_buffer.py	141		COMMENT
LOW	eden/integration/projfs_buffer.py	161	# \| full file stopped \| \| \| \| x \| \| \| x \| -	COMMENT
LOW	eden/integration/privhelper_test.py	101	# EdenFS cannot unmount its checkouts, so the daemon exits non-zero.	COMMENT
LOW	eden/integration/takeover_test.py	101		COMMENT
LOW	eden/integration/takeover_test.py	161	# even though we deleted the file from the tree	COMMENT
LOW	eden/integration/snapshot/snapshot.py	321	# looks like the old path. For now this is the easiest thing to do, and the	COMMENT
LOW	eden/integration/fsck/basic_snapshot_tests.py	641	# TODO: replace untracked dir with file	COMMENT
LOW	eden/integration/lib/ntapi.cpp	1	/*	COMMENT
LOW	eden/integration/lib/skip.py	1	#!/usr/bin/env python3	COMMENT
LOW	eden/integration/lib/testcase.py	661	enable_logview: bool = False	COMMENT
LOW	eden/integration/lib/service_test_case.py	141	# given input test class.	COMMENT
LOW	eden/integration/hg/rebase_test.py	81	#	COMMENT
LOW	eden/integration/hg/lib/hg_extension_test_base.py	561	# not a fixed string.	COMMENT
LOW	eden/integration/helpers/fake_edenfs.cpp	1	/*	COMMENT
LOW	eden/integration/helpers/fake_edenfs.cpp	21	#include <iterator>	COMMENT
LOW	eden/contrib/shared/textmate-lib/textmateStyles.ts	41	// theme globally on the page at a time, at least for now.)	COMMENT
LOW	eden/contrib/shared/__tests__/diff.test.ts	41	// cases = []	COMMENT
LOW	eden/contrib/reviewstack/src/ActorAvatar.tsx	21	const noFallback = useAtomValue(isConsumerGitHubAtom);	COMMENT
LOW	eden/contrib/reviewstack/src/PullRequestStack.tsx	21		COMMENT
LOW	…/contrib/reviewstack/src/github/GraphQLGitHubClient.ts	201	decodedContent != null && !isBinary	COMMENT
LOW	eden/mononoke/clients/scsc/src/connection.rs	41		COMMENT
LOW	…e/clients/scsc/src/commands/commit_rate_limit_check.rs	21	#[derive(clap::Parser)]	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/export.rs	101	output: String,	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/ls.rs	41		COMMENT
LOW	…en/mononoke/clients/scsc/src/commands/move_bookmark.rs	21	/// Move a bookmark	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/log.rs	41	];	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/log.rs	61	#[clap(flatten)]	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/xrepo_lookup.rs	41	source_repo: String,	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/create_repos.rs	21	#[clap(long)]	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/run_hooks.rs	41	#[clap(flatten)]	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/run_hooks.rs	61	/// Like --run-as, but takes a percent-encoded JSON identity envelope (the	COMMENT
LOW	…n/mononoke/clients/scsc/src/commands/list_bookmarks.rs	41	scheme_args: SchemeArgs,	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/blame.rs	41	repo_args: RepoArgs,	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/blame.rs	61	#[clap(short = 'q')]	COMMENT
LOW	…onoke/clients/scsc/src/commands/scmqueryclient_test.rs	1	/*	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/diff.rs	41	pub(super) struct CommandArgs {	COMMENT
LOW	eden/mononoke/clients/scsc/src/commands/diff.rs	61	#[clap(long, short = 'O')]	COMMENT
LOW	…clients/scsc/src/commands/directory_branch_clusters.rs	21	use crate::render::Render;	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	141	/// Hipster tier that is permitted to act as a trusted proxy.	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	181	/// Configuration of a single repository	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	201	/// Infinitepush configuration	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	241	pub repo_client_use_warm_bookmarks_cache: bool,	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	261	/// on select servers.	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	281	/// Sets a multiplier for the value for the objects count metric for this repo	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	301	/// endpoint.	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	561	}	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	661	pub max_inlined_delta_size: u64,	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	861	pub bookmark: BookmarkOrRegex,	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	1041		COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	1081	/// Per-request override for the `pushrebase_enable_merge_resolution` JustKnob.	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	1341	pub struct StorageConfig {	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	1381	/// Configuration for a blobstore	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	1681	/// No marking or deletion	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	1761	pub enum DefaultSmallToLargeCommitSyncPathAction {	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	1781	/// repo into the synced version of the source repo in the target repo.	COMMENT
LOW	eden/mononoke/metaconfig/types/src/lib.rs	1801	/// as file changes from the submodule dependencies might need to be copied.	COMMENT
1937 more matches not shown…

Empty Error Handlers344 hits · 1720 pts

Severity	File	Line	Snippet	Context
HIGH	eden/integration/lock_test.py	18	except ImportError:	CODE
HIGH	eden/integration/lock_test.py	209	except AssertionError:	CODE
HIGH	eden/integration/lock_test.py	239	except AssertionError:	CODE
HIGH	eden/integration/projfs_enumeration.py	21	except ImportError:	CODE
HIGH	eden/integration/eden_lock_test.py	18	except ImportError:	CODE
HIGH	eden/integration/snapshot/inode_metadata.py	161	except Exception:	CODE
HIGH	eden/integration/lib/skip.py	462	except ImportError:	CODE
HIGH	eden/integration/lib/hgrepo.py	312	except FileExistsError:	CODE
HIGH	eden/integration/lib/edenclient.py	110	except (IndexError, OSError):	CODE
HIGH	eden/integration/lib/edenclient.py	818	except Exception:	CODE
HIGH	eden/integration/hg/update_test.py	1563	except Exception:	CODE
HIGH	eden/integration/hg/update_test.py	1597	except Exception:	CODE
HIGH	eden/integration/hg/update_test.py	1638	except Exception:	CODE
HIGH	eden/integration/hg/update_test.py	1696	except Exception:	CODE
HIGH	eden/integration/hg/update_test.py	881	except Exception:	CODE
HIGH	eden/scm/build.py	268	except Exception:	CODE
HIGH	eden/scm/build.py	252	except Exception:	CODE
HIGH	eden/scm/build.py	504	except OSError:	CODE
HIGH	eden/scm/ghstack/logs.py	160	except FileExistsError:	CODE
HIGH	eden/scm/ghstack/logs.py	177	except FileExistsError:	CODE
HIGH	eden/scm/tests/get-with-headers.py	30	except ImportError:	CODE
HIGH	eden/scm/tests/sortdictfilter.py	30	except Exception:	CODE
HIGH	eden/scm/tests/printenv.py	36	except ImportError:	CODE
HIGH	eden/scm/tests/stresstest-condint.py	98	except ThreadInterrupt:	CODE
HIGH	eden/scm/tests/stresstest-condint.py	81	except ThreadInterrupt:	CODE
HIGH	eden/scm/tests/hghave.py	332	except OSError:	CODE
HIGH	eden/scm/tests/logexceptions.py	52	except (IndexError, OSError):	CODE
HIGH	eden/scm/tests/sitecustomize.py	17	except ImportError:	CODE
HIGH	eden/scm/tests/test-manifest-insert-before-remove.py	44	except RuntimeError:	CODE
HIGH	eden/scm/tests/run-tests.py	119	except NameError:	CODE
HIGH	eden/scm/tests/run-tests.py	144	except ImportError:	CODE
HIGH	eden/scm/tests/run-tests.py	808	except Exception:	CODE
HIGH	eden/scm/tests/run-tests.py	3042	except OSError:	CODE
HIGH	eden/scm/tests/run-tests.py	3046	except OSError:	CODE
HIGH	eden/scm/tests/run-tests.py	4332	except ImportError:	CODE
HIGH	eden/scm/tests/run-tests.py	983	except OSError:	CODE
HIGH	eden/scm/tests/run-tests.py	1409	except Exception:	CODE
HIGH	eden/scm/tests/run-tests.py	2856	except KeyboardInterrupt:	CODE
HIGH	eden/scm/tests/run-tests.py	3972	except OSError:	CODE
HIGH	eden/scm/tests/run-tests.py	2864	except KeyError:	CODE
HIGH	eden/scm/tests/run-tests.py	2868	except IndexError:	CODE
HIGH	eden/scm/tests/run-tests.py	1047	except OSError:	CODE
HIGH	eden/scm/tests/test-linelog-edits.py	54	except Exception:	CODE
HIGH	eden/scm/tests/test-filecache.py	31	except AttributeError:	CODE
HIGH	eden/scm/tests/killdaemons.py	133	except IOError:	CODE
HIGH	eden/scm/tests/test-atomictempfile.py	124	except ValueError:	CODE
HIGH	eden/scm/sapling/alerts.py	39	except Exception:	CODE
HIGH	eden/scm/sapling/mail.py	239	except UnicodeDecodeError:	CODE
HIGH	eden/scm/sapling/mail.py	385	except UnicodeDecodeError:	CODE
HIGH	eden/scm/sapling/mail.py	380	except UnicodeDecodeError:	CODE
HIGH	eden/scm/sapling/mail.py	295	except UnicodeEncodeError:	CODE
HIGH	eden/scm/sapling/worker.py	31	except (AttributeError, ValueError):	CODE
HIGH	eden/scm/sapling/worker.py	39	except (KeyError, ValueError):	CODE
HIGH	eden/scm/sapling/signing.py	381	except OSError:	CODE
HIGH	eden/scm/sapling/store.py	338	except Exception:	CODE
HIGH	eden/scm/sapling/store.py	191	except KeyError:	CODE
HIGH	eden/scm/sapling/git.py	1447	except ValueError:	CODE
HIGH	eden/scm/sapling/git.py	1466	except ValueError:	CODE
HIGH	eden/scm/sapling/git.py	1472	except ValueError:	CODE
HIGH	eden/scm/sapling/server.py	89	except (AttributeError, OSError):	CODE
284 more matches not shown…

Hyper-Verbose Identifiers1732 hits · 1591 pts

Severity	File	Line	Snippet	Context
LOW	make-client.py	109	def move_site_packages_to_root(instdir):	CODE
LOW	eden/integration/fifo_test.py	57	def test_fifo_persists_across_restart(self) -> None:	CODE
LOW	eden/integration/fifo_test.py	82	def test_disable_fifo_creation(self) -> None:	CODE
LOW	eden/integration/casing_test.py	33	def test_case_preserving_new_files(self) -> None:	CODE
LOW	eden/integration/use_case_test.py	33	def test_eden_cli_use_case_returns_without_error_bad_use_case(self) -> None:	CODE
LOW	eden/integration/invalidate_test.py	160	async def test_no_invalidation_with_age(self) -> None:	CODE
LOW	eden/integration/invalidate_test.py	207	async def test_partial_directory_invalidate(self) -> None:	CODE
LOW	eden/integration/invalidate_test.py	229	async def test_invalidate_background(self) -> None:	CODE
LOW	eden/integration/invalidate_test.py	235	async def test_invalidate_keep_timestamp(self) -> None:	CODE
LOW	eden/integration/du_test.py	32	def test_du_basic_runs_successfully(self) -> None:	CODE
LOW	eden/integration/du_test.py	44	def test_du_json_output_has_valid_fields(self) -> None:	CODE
LOW⚡	eden/integration/du_test.py	68	def test_du_fast_mode_shows_summary_only(self) -> None:	CODE
LOW⚡	eden/integration/du_test.py	78	def test_du_fast_and_json_report_same_base_values(self) -> None:	CODE
LOW⚡	eden/integration/du_test.py	88	def test_du_clean_mode_shows_warning_and_sections(self) -> None:	CODE
LOW	eden/integration/du_test.py	106	def test_du_deep_clean_mode_shows_warning(self) -> None:	CODE
LOW	eden/integration/du_test.py	118	def test_du_clean_orphaned_mode_runs_successfully(self) -> None:	CODE
LOW	eden/integration/du_test.py	124	def test_du_materialized_increases_after_file_modification(self) -> None:	CODE
LOW	eden/integration/du_test.py	144	def test_du_ignored_counts_untracked_files(self) -> None:	CODE
LOW	eden/integration/du_test.py	164	def test_du_materialized_increases_with_new_directory(self) -> None:	CODE
LOW	eden/integration/wal_test.py	173	def test_wal_replay_after_restart(self) -> None:	CODE
LOW	eden/integration/wal_test.py	203	def test_wal_torn_write_recovery(self) -> None:	CODE
LOW	eden/integration/wal_test.py	233	def test_fsck_replays_orphan_wal(self) -> None:	CODE
LOW	eden/integration/wal_test.py	270	def test_wal_concurrent_writers(self) -> None:	CODE
LOW	eden/integration/thrift_test.py	284	async def test_get_digest_hash_for_file(self) -> None:	CODE
LOW	eden/integration/thrift_test.py	302	async def test_get_digest_hash_for_directory(self) -> None:	CODE
LOW	eden/integration/thrift_test.py	350	async def test_get_digest_hash_throws_for_path_with_dot_components(self) -> None:	CODE
LOW	eden/integration/thrift_test.py	384	async def test_get_digest_hash_throws_for_empty_string(self) -> None:	CODE
LOW	eden/integration/thrift_test.py	447	async def test_get_digest_hash_throws_for_non_existent_file(self) -> None:	CODE
LOW	eden/integration/thrift_test.py	488	async def test_get_digest_hash_throws_for_materialized_directory(self) -> None:	CODE
LOW	eden/integration/thrift_test.py	511	async def test_get_digest_hash_throws_for_local_directory(self) -> None:	CODE
LOW	eden/integration/thrift_test.py	633	async def test_unload_thrift_api_accepts_single_dot_as_root(self) -> None:	CODE
LOW	eden/integration/thrift_test.py	688	async def test_diff_revisions_with_reverted_file(self) -> None:	CODE
LOW	eden/integration/notify_test.py	241	async def test_debug_subscribe_with_states(self) -> None:	CODE
LOW	eden/integration/notify_test.py	385	async def test_debug_subscribe_with_multiple_states(self) -> None:	CODE
LOW	eden/integration/notify_test.py	556	async def test_subscribe_unpack_commit_transitions(self) -> None:	CODE
LOW	eden/integration/notify_test.py	663	async def test_multiple_state_enter(self) -> None:	CODE
LOW	eden/integration/notify_test.py	681	async def test_subscribe_cleanup_left_over_state(self) -> None:	CODE
LOW	eden/integration/notify_test.py	715	async def test_subscribe_max_events(self) -> None:	CODE
LOW	eden/integration/notify_test.py	746	async def test_subscribe_cleanup_left_over_state_multiple_checks(self) -> None:	CODE
LOW	eden/integration/telemetry_test.py	40	async def test_harness_disables_scribe_logging(self) -> None:	CODE
LOW	eden/integration/telemetry_test.py	51	async def test_no_scuba_samples_are_enqueued(self) -> None:	CODE
LOW	eden/integration/remove_test.py	66	def test_remove_checkout_with_redirections(self, impl: str, env: dict) -> None:	CODE
LOW	eden/integration/remove_test.py	117	def test_remove_aux_process_timeout(self, impl: str, env: dict) -> None:	CODE
LOW	eden/integration/remove_test.py	201	def test_remove_with_busy_bind_mount(self, impl: str, env: dict) -> None:	CODE
LOW	eden/integration/remove_test.py	287	def test_rust_remove_succeeds_when_configured_client_dir_is_missing(self) -> None:	CODE
LOW	eden/integration/prjfs_match_fs.py	91	async def test_fix_missed_directory_delete(self) -> None:	CODE
LOW	eden/integration/prjfs_match_fs.py	108	async def test_fix_missed_directory_addition(self) -> None:	CODE
LOW	eden/integration/clone_test.py	35	def test_clone_to_non_existent_directory(self) -> None:	CODE
LOW	eden/integration/clone_test.py	45	async def test_clone_to_dir_under_symlink(self) -> None:	CODE
LOW	eden/integration/clone_test.py	71	def test_clone_to_existing_empty_directory(self) -> None:	CODE
LOW	eden/integration/clone_test.py	92	def test_clone_with_arcconfig(self) -> None:	CODE
LOW	eden/integration/clone_test.py	377	def test_clone_blocked_when_at_limit(self) -> None:	CODE
LOW	eden/integration/clone_test.py	407	def test_clone_allowed_when_under_limit(self) -> None:	CODE
LOW	eden/integration/clone_test.py	418	def test_python_clone_with_missing_configured_checkout_state_succeeds(self) -> None:	CODE
LOW	eden/integration/clone_test.py	497	def test_daemon_command_arguments_should_forward_to_edenfs(self) -> None:	CODE
LOW⚡	eden/integration/clone_test.py	188	def test_clone_with_valid_revision_cmd_line_arg_works(self) -> None:	STRING
LOW⚡	eden/integration/clone_test.py	197	def test_clone_with_short_revision_cmd_line_arg_works(self) -> None:	STRING
LOW⚡	eden/integration/clone_test.py	207	def test_clone_to_non_empty_directory_fails(self) -> None:	STRING
LOW⚡	eden/integration/clone_test.py	333	def test_default_case_sensitivity(self) -> None:	STRING
LOW⚡	eden/integration/clone_test.py	343	def test_force_case_sensitive(self) -> None:	STRING
1672 more matches not shown…

Deep Nesting756 hits · 667 pts

Severity	File	Line	Context
LOW	make-client.py	81	CODE
LOW	eden/integration/edenclient_test.py	164	CODE
LOW	eden/integration/debug_test.py	105	CODE
LOW	eden/integration/testcase_test.py	375	CODE
LOW	eden/integration/stop_test.py	43	CODE
LOW	eden/integration/materialized_query_test.py	55	CODE
LOW	eden/integration/takeover_test.py	479	CODE
LOW	eden/integration/lib/testcase.py	758	CODE
LOW	eden/integration/lib/testcase.py	959	CODE
LOW	eden/integration/lib/thrift_objects.py	48	CODE
LOW	eden/integration/lib/journal_test_base.py	95	CODE
LOW	eden/integration/hg/status_test.py	314	CODE
LOW	eden/integration/hg/lib/hg_extension_test_base_test.py	161	CODE
LOW	eden/mononoke/tests/integration/run_tests_getdeps.py	126	CODE
LOW	eden/mononoke/tests/utils/generate_buck_target_deps.py	39	CODE
LOW	eden/mononoke/scripts/merge_resolution_blame_checker.py	287	CODE
LOW	eden/mononoke/linters/jk_exists_linter.py	65	CODE
LOW	eden/scm/build.py	75	CODE
LOW	eden/scm/build.py	496	CODE
LOW	eden/scm/build.py	580	CODE
LOW	eden/scm/packaging/windows/build_windows_zip.py	93	CODE
LOW	eden/scm/packaging/windows/build_windows_zip.py	111	CODE
LOW	eden/scm/ghstack/submit.py	1036	CODE
LOW	eden/scm/ghstack/shell.py	104	CODE
LOW	eden/scm/ghstack/shell.py	157	CODE
LOW	eden/scm/ghstack/rage.py	31	CODE
LOW	eden/scm/ghstack/sapling_shell.py	173	CODE
LOW	eden/scm/ghstack/sapling_shell.py	59	CODE
LOW	eden/scm/ghstack/unlink.py	28	CODE
LOW	eden/scm/tests/test-check-win32-signature.py	10	CODE
LOW	eden/scm/tests/sortdictfilter.py	14	CODE
LOW	eden/scm/tests/edenfs.py	65	CODE
LOW	eden/scm/tests/stresstest-condint.py	45	CODE
LOW	eden/scm/tests/stresstest-condint.py	103	CODE
LOW	eden/scm/tests/test-revlog-raw.py	296	CODE
LOW	eden/scm/tests/test-rustthreading.py	92	CODE
LOW	eden/scm/tests/fix-revnum.py	54	CODE
LOW	eden/scm/tests/generateworkingcopystates.py	71	CODE
LOW	eden/scm/tests/run-tests.py	372	CODE
LOW	eden/scm/tests/run-tests.py	1284	CODE
LOW	eden/scm/tests/run-tests.py	1948	CODE
LOW	eden/scm/tests/run-tests.py	1962	CODE
LOW	eden/scm/tests/run-tests.py	2144	CODE
LOW	eden/scm/tests/run-tests.py	2250	CODE
LOW	eden/scm/tests/run-tests.py	2560	CODE
LOW	eden/scm/tests/run-tests.py	2613	CODE
LOW	eden/scm/tests/run-tests.py	2628	CODE
LOW	eden/scm/tests/run-tests.py	2788	CODE
LOW	eden/scm/tests/run-tests.py	3087	CODE
LOW	eden/scm/tests/run-tests.py	3303	CODE
LOW	eden/scm/tests/run-tests.py	3343	CODE
LOW	eden/scm/tests/run-tests.py	3445	CODE
LOW	eden/scm/tests/run-tests.py	3479	CODE
LOW	eden/scm/tests/run-tests.py	3762	CODE
LOW	eden/scm/tests/run-tests.py	3886	CODE
LOW	eden/scm/tests/run-tests.py	3975	CODE
LOW	eden/scm/tests/run-tests.py	4031	CODE
LOW	eden/scm/tests/edit-feature-header.py	57	CODE
LOW	eden/scm/tests/edit-feature-header.py	97	CODE
LOW	eden/scm/tests/test-ancestor.py	18	CODE
696 more matches not shown…

Excessive Try-Catch Wrapping510 hits · 534 pts

Severity	File	Line	Snippet	Context
LOW⚡	eden/integration/prjfs_stress.py	324	except Exception:	CODE
LOW⚡	eden/integration/takeover_test.py	405	except Exception:	CODE
LOW⚡	eden/integration/takeover_test.py	420	except Exception:	CODE
LOW	eden/integration/snapshot/inode_metadata.py	158	except Exception:	CODE
LOW	eden/integration/snapshot/inode_metadata.py	161	except Exception:	CODE
LOW	eden/integration/snapshot/snapshot.py	585	except Exception:	CODE
LOW	eden/integration/lib/testcase.py	154	except Exception as e:	CODE
LOW	eden/integration/lib/edenclient.py	747	except Exception:	CODE
LOW	eden/integration/lib/edenclient.py	818	except Exception:	CODE
LOW	eden/integration/hg/status_test.py	546	except Exception as e:	CODE
LOW	eden/integration/hg/status_test.py	603	except Exception as e:	CODE
LOW⚡	eden/integration/hg/filteredhg_migration_test.py	406	except Exception:	STRING
LOW	eden/integration/hg/filteredhg_migration_test.py	503	except Exception:	STRING
MEDIUM	eden/integration/hg/update_test.py	507	def checkout_in_progress() -> Optional[bool]:	CODE
MEDIUM	eden/integration/hg/update_test.py	973	def update_to_bottom() -> None:	CODE
LOW	eden/integration/hg/update_test.py	554	except Exception as ex:	CODE
LOW	eden/integration/hg/update_test.py	881	except Exception:	CODE
LOW	eden/integration/hg/update_test.py	1554	except Exception as err:	CODE
LOW	eden/integration/hg/update_test.py	1563	except Exception:	CODE
LOW	eden/integration/hg/update_test.py	1597	except Exception:	CODE
LOW	eden/integration/hg/update_test.py	1629	except Exception as err:	CODE
LOW	eden/integration/hg/update_test.py	1638	except Exception:	CODE
LOW	eden/integration/hg/update_test.py	1655	except Exception:	CODE
LOW	eden/integration/hg/update_test.py	1685	except Exception as err:	CODE
LOW	eden/integration/hg/update_test.py	1696	except Exception:	CODE
LOW	eden/mononoke/scripts/canary_shard_pin.py	162	except Exception as e:	CODE
LOW	eden/mononoke/scripts/canary_shard_pin.py	189	except Exception as e:	CODE
LOW	eden/mononoke/scripts/merge_resolution_blame_checker.py	154	except Exception:	STRING
LOW	eden/mononoke/scripts/merge_resolution_blame_checker.py	325	except Exception:	STRING
LOW	eden/mononoke/scripts/merge_resolution_blame_checker.py	369	except Exception:	STRING
LOW	eden/scm/build.py	199	except Exception:	CODE
LOW	eden/scm/build.py	227	except Exception:	CODE
LOW	eden/scm/build.py	252	except Exception:	CODE
LOW	eden/scm/build.py	268	except Exception:	CODE
LOW	eden/scm/build.py	611	except Exception:	CODE
MEDIUM	eden/scm/build.py	262	def rustup_override():	CODE
LOW	eden/scm/ghstack/logs.py	78	except Exception as e:	CODE
LOW	eden/scm/hgcache_size_logger/hgcache_size_logger.py	29	except Exception as e:	CODE
LOW	eden/scm/hgcache_size_logger/hgcache_size_logger.py	71	except Exception as exc:	CODE
LOW	eden/scm/hgcache_size_logger/hgcache_size_logger.py	77	except Exception:	CODE
LOW	eden/scm/tests/heredoctest.py	20	except Exception as inst:	CODE
LOW	eden/scm/tests/sortdictfilter.py	30	except Exception:	CODE
LOW	eden/scm/tests/edenfs.py	164	except Exception as e:	STRING
LOW	eden/scm/tests/test-nested-lock.py	25	except Exception as ex:	CODE
LOW	eden/scm/tests/test-revlog-raw.py	90	except Exception as ex:	CODE
LOW	eden/scm/tests/stableidentifiers.py	20	except Exception:	CODE
LOW	eden/scm/tests/test-rustthreading.py	109	except Exception:	CODE
LOW	eden/scm/tests/test-rustthreading.py	151	except Exception:	CODE
LOW	eden/scm/tests/hghave.py	93	except Exception:	CODE
MEDIUM	eden/scm/tests/run-tests.py	795	def try_increase_open_file_limit():	CODE
LOW	eden/scm/tests/run-tests.py	76	except Exception:	CODE
LOW	eden/scm/tests/run-tests.py	808	except Exception:	CODE
LOW	eden/scm/tests/run-tests.py	1242	except Exception:	CODE
LOW	eden/scm/tests/run-tests.py	1265	except Exception:	CODE
LOW	eden/scm/tests/run-tests.py	1275	except Exception:	CODE
LOW	eden/scm/tests/run-tests.py	1381	except Exception as e:	CODE
LOW	eden/scm/tests/run-tests.py	1409	except Exception:	CODE
LOW	eden/scm/tests/run-tests.py	1888	except Exception:	CODE
LOW	eden/scm/tests/run-tests.py	2925	except Exception:	CODE
LOW	eden/scm/tests/test-context.py	173	except Exception as ex:	CODE
450 more matches not shown…

Cross-Language Confusion90 hits · 533 pts

Severity	File	Line	Snippet	Context
HIGH	eden/integration/thrift_test.py	118	self.repo.push(rev=".", target="master", create=True)	CODE
HIGH⚡	eden/integration/start_test.py	438	# pyre-fixme[61]: `index` is undefined, or not always defined.	COMMENT
HIGH⚡	eden/integration/mmap_test.py	59	# assert the additional mapped bytes are null, per `man 2 mmap`	COMMENT
HIGH	eden/integration/mmap_test.py	115	# Assert the additional mapped bytes are null, per `man 2 mmap`	COMMENT
HIGH	eden/integration/readdir_test.py	184	self.repo.push(rev=".", target="master", create=True)	CODE
HIGH	eden/integration/xattr_test.py	48	self.repo.push(".", "master", create=True)	CODE
HIGH⚡	eden/scm/build.py	306	rustflags = ["-C", "link-args=-Wl,-undefined,dynamic_lookup"]	CODE
HIGH⚡	eden/scm/build.py	309	rustflags = ["-C", "link-args=-Wl,-undefined,dynamic_lookup"]	CODE
HIGH⚡	eden/scm/build.py	312	rustflags = ["-C", "link-args=-Wl,-undefined,dynamic_lookup"]	CODE
HIGH⚡	eden/scm/build.py	315	rustflags = ["-C", "link-args=-Wl,-undefined,dynamic_lookup"]	CODE
HIGH	eden/scm/gen_version.py	71	# pyre-fixme[61]: `contents` is undefined, or not always defined.	STRING
HIGH	eden/scm/tests/run-tests.py	2116	l = b" $ cd %s \|\| exit 1\n" % cmd[1]	CODE
HIGH	eden/scm/sapling/patch.py	1950	# not /dev/null, but afile starts with bfile	COMMENT
HIGH⚡	eden/scm/sapling/patch.py	2140	lr.push(l2)	CODE
HIGH⚡	eden/scm/sapling/patch.py	2150	lr.push(l2)	CODE
HIGH⚡	eden/scm/sapling/patch.py	2153	lr.push(l3)	CODE
HIGH⚡	eden/scm/sapling/patch.py	2155	lr.push(l2)	CODE
HIGH	eden/scm/sapling/patch.py	1487	lr.push(l)	CODE
HIGH	eden/scm/sapling/patch.py	1528	lr.push(l)	CODE
HIGH	eden/scm/sapling/patch.py	1537	lr.push(l)	CODE
HIGH	eden/scm/sapling/patch.py	1581	lr.push(l)	CODE
HIGH	eden/scm/sapling/patch.py	2014	lr.push(line)	CODE
HIGH	eden/scm/sapling/patch.py	2031	lr.push(fromfile)	CODE
HIGH	eden/scm/sapling/patch.py	2066	gitlr.push(firstline)	CODE
HIGH	eden/scm/sapling/merge.py	2354	# If we're doing the initial checkout from null, let's use the new fancier	COMMENT
HIGH	eden/scm/sapling/destutil.py	160	histeditdefaultrevset = "reverse(only(.) - public() - ::merge() - null)"	CODE
HIGH⚡	eden/scm/sapling/util.py	4112	>>> r.push(1)	STRING
HIGH⚡	eden/scm/sapling/util.py	4113	>>> r.push(2)	STRING
HIGH⚡	eden/scm/sapling/util.py	4114	>>> r.push(3)	STRING
HIGH⚡	eden/scm/sapling/util.py	4115	>>> r.push(4)	STRING
HIGH⚡	eden/scm/sapling/util.py	4124	>>> r.push(5)	STRING
HIGH⚡	eden/scm/sapling/util.py	4125	>>> r.push(6)	STRING
HIGH⚡	eden/scm/sapling/util.py	4126	>>> r.push(7)	STRING
HIGH	eden/scm/sapling/cmdutil.py	1108	'x [3, null]'	STRING
HIGH	eden/scm/sapling/mdiff.py	297	If the diff is not null, `headers` is a list with unified diff header	STRING
HIGH	eden/scm/sapling/revset.py	260	# special case: null::ys == ::ys (rust dag does not have null)	COMMENT
HIGH	eden/scm/sapling/revset.py	499	# special case: ancestors(null) == null (used by log -fr null)	COMMENT
HIGH	eden/scm/sapling/revset.py	1019	# special case: null::heads == ::heads (rust dag does not have null)	COMMENT
HIGH	eden/scm/sapling/localrepo.py	3153	ret = pushkey.push(self, namespace, key, old, new)	CODE
HIGH	eden/scm/sapling/revlog.py	131	# As of now, if one of the parent node is null, p2 is null	COMMENT
HIGH	eden/scm/sapling/sslutil.py	927	# pyre-fixme[61]: `hash` is undefined, or not always defined.	COMMENT
HIGH	eden/scm/sapling/helptext.py	4602	+ if (ignore_funny && 45 < len && !memcmp(name, "refs/", 5) &&	CODE
HIGH	eden/scm/sapling/helptext.py	4608	+ if (ignore_funny && 45 < len && !memcmp(name, "refs/", 5) &&	CODE
HIGH	eden/scm/sapling/helptext.py	4623	if (ignore_funny && 45 < len && !memcmp(name, "refs/", 5) &&	CODE
HIGH	eden/scm/sapling/lsprof.py	141	# pyre-fixme[61]: `mname` is undefined, or not always defined.	COMMENT
HIGH	eden/scm/sapling/bookmarks.py	897	# pyre-fixme[61]: `tgt` is undefined, or not always defined.	COMMENT
HIGH	eden/scm/sapling/hg.py	360	# pyre-fixme[61]: `uprev` is undefined, or not always defined.	COMMENT
HIGH	eden/scm/sapling/hg.py	594	exchange.push(	CODE
HIGH	eden/scm/sapling/hg.py	641	# pyre-fixme[61]: `srcpeer` is undefined, or not always defined.	COMMENT
HIGH	eden/scm/sapling/hg.py	648	# pyre-fixme[61]: `srcpeer` is undefined, or not always defined.	COMMENT
HIGH	eden/scm/sapling/progress.py	414	ring.push((pos, now))	CODE
HIGH	eden/scm/sapling/ext/rebase.py	2479	# better than the default (ancestor(F, Z) == null). Therefore still	COMMENT
HIGH	eden/scm/sapling/ext/tweakdefaults.py	385	# pyre-fixme[61]: `rebasemodule` is undefined, or not always defined.	COMMENT
HIGH	eden/scm/sapling/ext/tweakdefaults.py	387	# pyre-fixme[61]: `rebasemodule` is undefined, or not always defined.	COMMENT
HIGH⚡	eden/scm/sapling/ext/interactiveui.py	89	# pyre-fixme[61]: `ch` is undefined, or not always defined.	COMMENT
HIGH⚡	eden/scm/sapling/ext/interactiveui.py	92	# pyre-fixme[61]: `ch` is undefined, or not always defined.	COMMENT
HIGH⚡	eden/scm/sapling/ext/interactiveui.py	95	# pyre-fixme[61]: `ch` is undefined, or not always defined.	COMMENT
HIGH	eden/scm/sapling/ext/remotenames.py	870	return git.push(repo, dest, [(pushnode, to)], force=force)	CODE
HIGH	eden/scm/sapling/ext/remotenames.py	1032	return pushmod.push(	CODE
HIGH	eden/scm/sapling/ext/remotenames.py	1061	pushop = exchange.push(	CODE
30 more matches not shown…

Unused Imports340 hits · 318 pts

Severity	File	Line	Context
LOW	eden/integration/fifo_test.py	7	CODE
LOW	eden/integration/use_case_test.py	9	CODE
LOW	eden/integration/wal_test.py	19	CODE
LOW	eden/integration/mount_test.py	7	CODE
LOW	eden/integration/pressure_gc_test.py	7	CODE
LOW	eden/integration/testcase_test.py	7	CODE
LOW	eden/integration/materialized_query_test.py	7	CODE
LOW	eden/integration/rename_test.py	7	CODE
LOW	eden/integration/snapshot/test_snapshots.py	7	CODE
LOW	eden/integration/snapshot/snapshot.py	7	CODE
LOW	eden/integration/fsck/basic_snapshot_tests.py	7	CODE
LOW	eden/integration/lib/skip.py	345	CODE
LOW	eden/integration/lib/edenclient.py	7	CODE
LOW	eden/integration/hg/status_test.py	7	CODE
LOW	eden/integration/hg/filteredhg_migration_test.py	6	CODE
LOW	eden/integration/hg/lib/hg_extension_test_base_test.py	7	CODE
LOW	eden/test_support/temporary_directory.py	12	CODE
LOW	eden/test_support/temporary_directory.py	18	CODE
LOW	…/mononoke/tests/integration/integration_runner_real.py	34	CODE
LOW	eden/mononoke/scripts/canary_shard_pin.py	34	CODE
LOW	eden/mononoke/scripts/merge_resolution_blame_checker.py	21	CODE
LOW	eden/mononoke/linters/jk_exists_linter.py	20	CODE
LOW	eden/scm/tests/get-with-headers.py	14	CODE
LOW	eden/scm/tests/get-with-headers.py	14	CODE
LOW	eden/scm/tests/ls-l.py	9	CODE
LOW	eden/scm/tests/ls-l.py	9	CODE
LOW	eden/scm/tests/setconfig.py	6	CODE
LOW	eden/scm/tests/revnamesext.py	3	CODE
LOW	eden/scm/tests/common-pattern.py	2	CODE
LOW	eden/scm/tests/crashgetbundler.py	1	CODE
LOW	eden/scm/tests/test-sshserver.py	1	CODE
LOW	eden/scm/tests/test-sshserver.py	1	CODE
LOW	eden/scm/tests/heredoctest.py	1	CODE
LOW	eden/scm/tests/heredoctest.py	1	CODE
LOW	eden/scm/tests/test-lrucachefunc.py	6	CODE
LOW	eden/scm/tests/test-lrucachefunc.py	6	CODE
LOW	eden/scm/tests/stresstest-compress.py	11	CODE
LOW	eden/scm/tests/sortdictfilter.py	1	CODE
LOW	eden/scm/tests/test-zstdelta.py	1	CODE
LOW	eden/scm/tests/silenttestrunner.py	1	CODE
LOW	eden/scm/tests/silenttestrunner.py	1	CODE
LOW	eden/scm/tests/test-lrucachedict.py	1	CODE
LOW	eden/scm/tests/test-lrucachedict.py	1	CODE
LOW	eden/scm/tests/test-encoding-func.py	6	CODE
LOW	eden/scm/tests/test-symlink-os-yes-fs-no.py	1	CODE
LOW	eden/scm/tests/test-linerange.py	1	CODE
LOW	eden/scm/tests/test-wireproto.py	1	CODE
LOW	eden/scm/tests/test-wireproto.py	1	CODE
LOW	eden/scm/tests/flagprocessorext.py	11	CODE
LOW	eden/scm/tests/test-check-interfaces.py	3	CODE
LOW	eden/scm/tests/test-check-interfaces.py	3	CODE
LOW	eden/scm/tests/test-hg-parseurl.py	1	CODE
LOW	eden/scm/tests/test-hg-parseurl.py	1	CODE
LOW	eden/scm/tests/printenv.py	24	CODE
LOW	eden/scm/tests/test-status-inprocess.py	2	CODE
LOW	eden/scm/tests/test-status-inprocess.py	2	CODE
LOW	eden/scm/tests/test-revlog-raw.py	3	CODE
LOW	eden/scm/tests/test-revlog-raw.py	3	CODE
LOW	eden/scm/tests/test-rustthreading.py	50	CODE
LOW	eden/scm/tests/lockdelay.py	10	CODE
280 more matches not shown…

Self-Referential Comments97 hits · 284 pts

Severity	File	Line	Snippet	Context
MEDIUM	eden/integration/du_test.py	150	# Create a new untracked file	COMMENT
MEDIUM	eden/integration/du_test.py	170	# Create a new directory and file	COMMENT
MEDIUM	eden/integration/remove_test.py	127	# Create a second mount	COMMENT
MEDIUM	eden/integration/clone_test.py	142	# Create an Eden mount from the config alias.	STRING
MEDIUM	eden/integration/start_test.py	54	# Create a repository with one commit	COMMENT
MEDIUM	eden/integration/start_test.py	58	# Create an Eden checkout of this repository	COMMENT
MEDIUM	eden/integration/info_test.py	67	# Create a non-normalized symlink pointing to the parent directory	COMMENT
MEDIUM	eden/integration/mount_test.py	365	# Create a second checkout in a directory a couple levels deep so that	COMMENT
MEDIUM	eden/integration/redirect_test.py	649	# Create a bind mount that EdenFS is aware of but that is NOT present	COMMENT
MEDIUM⚡	eden/integration/debug_getpath_test.py	63	# Create the file	COMMENT
MEDIUM	eden/integration/debug_getpath_test.py	82	# Create the file	COMMENT
MEDIUM	eden/integration/debug_getpath_test.py	130	# Create the file	COMMENT
MEDIUM⚡	eden/integration/windows_fsck_test.py	64	# Create a file	COMMENT
MEDIUM⚡	eden/integration/windows_fsck_test.py	66	# Create a subdirectory	COMMENT
MEDIUM	eden/integration/windows_fsck_test.py	322	# Create a directory so the parent directory now becomes a DirtyPlaceholder	COMMENT
MEDIUM	eden/integration/windows_fsck_test.py	766	# Create a backup of the overlay. Later we'll restore this backup	COMMENT
MEDIUM	eden/integration/unlink_test.py	29	# This file is part of the git repo	COMMENT
MEDIUM	eden/integration/health_test.py	30	# Create a new edenfs instance that is never started, and make sure	COMMENT
MEDIUM	eden/integration/snapshot/test_snapshots.py	218	# Define a subclass of HgSnapshot. We use define this solely so we can use its	STRING
MEDIUM	eden/integration/snapshot/snapshot.py	410	# This file is not saved with the snapshot, and is instead regenerated each time	COMMENT
MEDIUM	eden/integration/snapshot/snapshot.py	421	# Create the repository	COMMENT
MEDIUM	eden/integration/snapshot/types/basic.py	80	# Create a new top-level directory with some new files	COMMENT
MEDIUM	eden/integration/lib/journal_test_base.py	44	# Create the initial repo. It requires at least 1 file and 1 commit	COMMENT
MEDIUM	eden/integration/lib/journal_test_base.py	348	# This class is intended to test the journal system for EdenFS on Windows.	COMMENT
MEDIUM	eden/integration/lib/hgrepo.py	207	# Create a temporary file for the input as a more reliable way than the PIPE	STRING
MEDIUM	eden/integration/hg/journal_test.py	29	# Create a new commit	COMMENT
MEDIUM	eden/integration/hg/graft_test.py	27	# Create a new head.	COMMENT
MEDIUM	eden/integration/hg/graft_test.py	50	# Create a new head.	COMMENT
MEDIUM⚡	eden/integration/hg/graft_test.py	82	# Create a new head that adds second.txt and removes first.txt.	COMMENT
MEDIUM⚡	eden/integration/hg/graft_test.py	88	# Create a separate head that adds third.txt.	COMMENT
MEDIUM⚡	eden/integration/hg/add_test.py	82	# Create an ordinary file with the same name as the directory that was	COMMENT
MEDIUM	eden/integration/hg/non_eden_operation_test.py	27	# Create the non-Eden Hg repo to clone.	COMMENT
MEDIUM	eden/integration/hg/status_test.py	952	# Create a commit with an executable file	COMMENT
MEDIUM	eden/integration/hg/status_test.py	1002	# Define a separate TestCase class purely to test with different initial	COMMENT
MEDIUM	eden/integration/hg/update_test.py	458	# Create a commit with a newly-created file foo/new_file.txt	COMMENT
MEDIUM	eden/integration/hg/update_test.py	646	# Create a new commit in the backing repository with many new files in	COMMENT
MEDIUM	eden/integration/hg/pull_test.py	23	# Create a server repository first	COMMENT
MEDIUM	eden/integration/hg/pull_test.py	41	# Create a server repository.	COMMENT
MEDIUM	eden/integration/hg/pull_test.py	48	# Create a commit in the server repository	COMMENT
MEDIUM	eden/integration/hg/pull_test.py	67	# Create a few new commits on the server	COMMENT
MEDIUM	eden/integration/hg/pull_test.py	97	# Create a 4th commit on the server	COMMENT
MEDIUM	eden/integration/hg/lib/hg_extension_test_base.py	90	# Create the backing repository	COMMENT
MEDIUM	eden/integration/hg/lib/hg_extension_test_base.py	220	# Create the HgRepository objects for the new mount and backing repo	COMMENT
MEDIUM	eden/test_support/temporary_directory.py	224	"""This class is largely equivalent to tempfile.NamedTemporaryFile,	STRING
MEDIUM	…ation/cross_repo/library-git-submodules-repos-setup.sh	263	# Create a commit in repo_b that can be used to update its submodule pointer	COMMENT
MEDIUM	…ation/cross_repo/library-git-submodules-repos-setup.sh	269	# Create a commit in repo_b to update its repo_a pointer from the large repo	COMMENT
MEDIUM	…ation/cross_repo/library-git-submodules-repos-setup.sh	290	# Create a commit in repo_b and update its submodule pointer from the large repo	COMMENT
MEDIUM	…ation/cross_repo/library-git-submodules-repos-setup.sh	301	# Create a commit in repo_a.	COMMENT
MEDIUM	…ation/cross_repo/library-git-submodules-repos-setup.sh	325	# Create a commit in repo_c that can be used to update its submodule pointer	COMMENT
MEDIUM	…ation/cross_repo/library-git-submodules-repos-setup.sh	330	# Create a commit in repo_b to update its repo_a pointer from the large repo	COMMENT
MEDIUM	…ation/cross_repo/library-git-submodules-repos-setup.sh	377	# Create a commit in repo_b and repo_c, then update its submodule pointers from	COMMENT
MEDIUM	…tion/cross_repo/library-git-submodules-config-setup.sh	178	# Create a commit in the large repo	COMMENT
MEDIUM	eden/scm/packaging/mac/prepare_formula.py	14	# Create the parser	COMMENT
MEDIUM	eden/scm/ghstack/submit.py	316	# Create the PR in draft mode if it is going to be created (and not updated).	COMMENT
MEDIUM	eden/scm/ghstack/submit.py	599	# Create the incremental pull request diff	COMMENT
MEDIUM	eden/scm/ghstack/diff.py	111	# This function is provided a shell whose cwd is the Git repository	COMMENT
MEDIUM	eden/scm/tests/tinit.sh	52	# Create a new repo	COMMENT
MEDIUM	eden/scm/tests/tinit.sh	326	# Create a new extension	COMMENT
MEDIUM	eden/scm/tests/run-tests.py	1231	# This function is extremely similar to unittest.TestCase.run(). Once	COMMENT
MEDIUM	eden/scm/tests/svn/svndump-branches.sh	47	# Create a cross-branch revision	COMMENT
37 more matches not shown…

Structural Annotation Overuse95 hits · 178 pts

Severity	File	Line	Snippet	Context
LOW⚡	eden/integration/mmap_test.py	69	# WARNING: This test is very fiddly.	COMMENT
LOW	…ols/admin/src/commands/derived_data/backfill_status.rs	460	// Step 1: Verify the backfill exists	COMMENT
LOW	…ols/admin/src/commands/derived_data/backfill_status.rs	489	// Step 2: Get aggregated stats	COMMENT
LOW⚡	…ols/admin/src/commands/derived_data/backfill_status.rs	503	// Step 3: Get timing stats	COMMENT
LOW⚡	…ols/admin/src/commands/derived_data/backfill_status.rs	509	// Step 4: Calculate metrics	COMMENT
LOW⚡	…rks/warm_bookmarks_cache/src/scoped_bookmarks_tests.rs	52	// Step 1: Derive only Hg data (and common dependencies), but not Git	COMMENT
LOW⚡	…rks/warm_bookmarks_cache/src/scoped_bookmarks_tests.rs	57	// Step 2: Initialize bookmarks - should include master since HgOnly is derived	COMMENT
LOW	…rks/warm_bookmarks_cache/src/scoped_bookmarks_tests.rs	87	// Step 3: Now derive Git data to make AllKinds ready	COMMENT
LOW	…rks/warm_bookmarks_cache/src/scoped_bookmarks_tests.rs	100	// Step 4: Verify bookmark now has all scopes tracked	COMMENT
LOW⚡	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	474	// Step 0: Validate if bubble deletion is enabled.	COMMENT
LOW⚡	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	478	// Step 1: Mark the bubble as expired in the backing SQL Store.	COMMENT
LOW⚡	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	492	// Step 2: Delete the blob content within the expired bubble.	COMMENT
LOW⚡	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	496	// Step 3: Delete the metadata associated with the bubble from	COMMENT
LOW	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	525	// Step 4: Delete the bubble itself from the backing SQL store.	COMMENT
LOW	…res/commit_transformation/src/git_submodules/expand.rs	878	// Step 1: Generate the submodule metadata file change	COMMENT
LOW	…res/commit_transformation/src/git_submodules/expand.rs	928	// Step 2: Generate the deletions of files/directories that are being	COMMENT
LOW	…/mononoke/docs/gitimport_from_bundle_pipeline_guide.md	55	## Step 1: Prepare the Git Bundle	COMMENT
LOW⚡	…/mononoke/docs/gitimport_from_bundle_pipeline_guide.md	73	## Step 2: Upload the Git Bundle to Manifold	COMMENT
LOW⚡	…/mononoke/docs/gitimport_from_bundle_pipeline_guide.md	104	## Step 3: Trigger the Skycastle Pipeline	COMMENT
LOW	…/mononoke/docs/gitimport_from_bundle_pipeline_guide.md	173	## Step 4: Check Pipeline Status	COMMENT
LOW⚡	…rived_data/directory_branch_cluster_manifest/derive.rs	426	// Step 1: Load parent manifests	COMMENT
LOW⚡	…rived_data/directory_branch_cluster_manifest/derive.rs	433	// Step 2: Merge parent manifests into a single base manifest	COMMENT
LOW⚡	…rived_data/directory_branch_cluster_manifest/derive.rs	436	// Step 3: Compute cluster changes using the merged parent for lookups	COMMENT
LOW	…rived_data/directory_branch_cluster_manifest/derive.rs	450	// Step 4: Apply cluster changes to the merged parent	COMMENT
LOW	eden/mononoke/common/mononoke_configs/src/lib.rs	563	// Step 1: Separate cached from missing	COMMENT
LOW	eden/mononoke/common/mononoke_configs/src/lib.rs	583	// Step 2: Subscribe to ConfigHandles + parse OUTSIDE rcu. Accumulate	COMMENT
LOW	eden/mononoke/common/mononoke_configs/src/lib.rs	595	// Step 3: rcu — bulk insert via a single closure that re-runs on CAS	COMMENT
LOW	…en/mononoke/common/async_limiter/src/async_limiter_.rs	31	// NOTE: This function is async because it requires a Tokio runtme to spawn things. The best	COMMENT
LOW⚡	eden/mononoke/scripts/merge_resolution_blame_checker.py	191	# Step 1: Find merge-resolved diffs	STRING
LOW	eden/mononoke/scripts/merge_resolution_blame_checker.py	212	# Step 2: Batch-check all diff IDs against trunk breakage blame	STRING
LOW	…s/skills/write-hook/references/implementation-guide.md	23	## Step 1: Decide hook type and config	COMMENT
LOW	…s/skills/write-hook/references/implementation-guide.md	33	## Step 2: Create 3 commits	COMMENT
LOW	…s/skills/write-hook/references/implementation-guide.md	61	## Step 3: Verify	COMMENT
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	442	// Step 1: Build partial commit graph (this exercises merge handling)	COMMENT
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	450	// Step 2: Rewrite changesets to temporary repo	COMMENT
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	460	// Step 3: Verify the rewritten repo has correct linear history	COMMENT
LOW	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	533	// Step 1: Build partial commit graph	COMMENT
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	545	// Step 2: Rewrite changesets to temporary repo	COMMENT
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	555	// Step 3: Verify the rewritten repo has a merge commit	COMMENT
LOW	eden/.llms/skills/CREATING_ENDPOINTS/SKILL.md	34	## Step 1: Define Request/Response Types	COMMENT
LOW	eden/.llms/skills/CREATING_ENDPOINTS/SKILL.md	106	## Step 2: Create Server-Side Handler	COMMENT
LOW	eden/.llms/skills/CREATING_ENDPOINTS/SKILL.md	228	## Step 3: Add Client-Side Implementation	COMMENT
LOW	…ms/skills/edenfs-systemd-triage/references/timeline.md	17	### Step 1: Gather Parameters	COMMENT
LOW	…ms/skills/edenfs-systemd-triage/references/timeline.md	24	### Step 2: Collect Events from All Sources	COMMENT
LOW	…ms/skills/edenfs-systemd-triage/references/timeline.md	127	### Step 3: Build the Timeline	COMMENT
LOW	…ms/skills/edenfs-systemd-triage/references/timeline.md	135	### Step 4: Separate Multiple EdenFS Instances	COMMENT
LOW	…ms/skills/edenfs-systemd-triage/references/timeline.md	146	### Step 5: Present the Timeline	COMMENT
LOW	…ms/skills/edenfs-systemd-triage/references/timeline.md	165	### Step 6: Annotate with Interpretation	COMMENT
LOW	…ls/edenfs-systemd-triage/references/triage-playbook.md	44	### Step 1: Check current state	COMMENT
LOW	…ls/edenfs-systemd-triage/references/triage-playbook.md	55	### Step 2: Check why the service failed	COMMENT
LOW	…ls/edenfs-systemd-triage/references/triage-playbook.md	70	### Step 3: If start-limit-hit, reset and restart	COMMENT
LOW	…ls/edenfs-systemd-triage/references/triage-playbook.md	79	### Step 4: Check if systemd-managed is enabled	COMMENT
LOW⚡	eden/scm/sapling/ext/snapshot/update.py	26	# Step 1: Forget files that were "hg added"	COMMENT
LOW⚡	eden/scm/sapling/ext/snapshot/update.py	33	# Step 2: Remove "untracked changes" (e.g. untracked files)	COMMENT
LOW⚡	eden/scm/sapling/ext/snapshot/update.py	42	# Step 3: Remove "tracked changes"	COMMENT
LOW⚡	eden/scm/lib/indexedlog/src/log.rs	775	// Step 4: Update the indexes and folds. Optionally flush them.	COMMENT
LOW⚡	eden/scm/lib/indexedlog/src/log.rs	784	// Step 5: Write the updated meta file.	COMMENT
LOW	eden/scm/lib/indexedlog/src/log.rs	628	// Step 1: Reload metadata to get the latest view of the files.	COMMENT
LOW	eden/scm/lib/indexedlog/src/log.rs	684	// Step 2: Append to the primary log.	COMMENT
LOW	eden/scm/lib/indexedlog/src/log.rs	744	// Step 3: Reload primary log and indexes to get the latest view.	COMMENT
35 more matches not shown…

Decorative Section Separators47 hits · 171 pts

Severity	File	Line	Snippet	Context
MEDIUM⚡	eden/mononoke/docs/local-dev-testing.md	170	# ┌──────────────────┬──────────────────┐	COMMENT
MEDIUM⚡	eden/mononoke/docs/local-dev-testing.md	172	# ├──────────────────┼──────────────────┤	COMMENT
MEDIUM⚡	eden/mononoke/docs/local-dev-testing.md	174	# └──────────────────┴──────────────────┘	COMMENT
MEDIUM	eden/mononoke/derived_data/blame/tests.rs	600	// ├────────────────────┐	COMMENT
MEDIUM	eden/mononoke/derived_data/blame/tests.rs	605	// └──────────┬──────────┘	COMMENT
MEDIUM	eden/scm/sapling/changelog2.py	705	# --------------------------------------------------	COMMENT
MEDIUM	eden/scm/sapling/ext/fsmonitor/__init__.py	140	# ===================	COMMENT
MEDIUM	eden/scm/sapling/ext/fsmonitor/__init__.py	155	# ============	COMMENT
MEDIUM	eden/scm/contrib/hg4idea_wrapper.py	53	# ----------------------------------------------------------------------	COMMENT
MEDIUM	eden/scm/contrib/hg4idea_wrapper.py	64	# ----------------------------------------------------------------------	COMMENT
MEDIUM	eden/scm/contrib/hg4idea_wrapper.py	439	# ----------------------------------------------------------------------	COMMENT
MEDIUM	eden/scm/contrib/hg4idea_wrapper.py	441	# ----------------------------------------------------------------------	COMMENT
MEDIUM	eden/scm/contrib/scm-prompt.sh	10	# ========================================================================	COMMENT
MEDIUM	eden/scm/contrib/scm-prompt.sh	75	# =========================================================================	COMMENT
MEDIUM	eden/scm/lib/linelog/src/debug.rs	111	// │ │ ╭────── Insert (rev x) <- this line	COMMENT
MEDIUM	eden/scm/lib/linelog/src/debug.rs	121	// Delete (rev x) ──────╮	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	196	// ┌──Row1──────────────────────────────┐	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	197	// │┌─Left──────────┐┌Right────────────┐│	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	198	// ││┌PreNode*─────┐││ ││	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	200	// ││├Node─────────┤││ ││	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	204	// ││└─────────────┘││ ││	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	205	// │└───────────────┘└─────────────────┘│	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	206	// └────────────────────────────────────┘	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	208	// ┌──Row2──────────────────────────────┐	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	209	// │┌─Left──────────┐┌Right────────────┐│	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	212	// ││├Term─────────┤││ ││	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	215	// ││├Padding──────┤││ ││	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	217	// ││├Link─────────┤││ ││	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	220	// ││├Ancestry─────┤││ ││	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	222	// │└───────────────┘└─────────────────┘│	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	223	// └────────────────────────────────────┘	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	236	// ┌──Row0──────────────────────────────┐	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	237	// │┌─Left─────────────┐ │	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	238	// ││┌Node────────────┐│ │	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	240	// ││└────────────────┘│ │	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	241	// │└──────────────────┘ │	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	242	// └────────────────────────────────────┘	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	243	// ┌──Row1──────────────────────────────┐	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	244	// │┌─Left──────────┐┌Right────────────┐│	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	247	// ││└─────────────┘││ ││	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	248	// │└───────────────┘└─────────────────┘│	COMMENT
MEDIUM⚡	addons/isl/src/RenderDag.tsx	249	// └────────────────────────────────────┘	COMMENT
MEDIUM	addons/isl/src/dag/render.ts	637	// │ ╰─────┤ │ ├───╯	COMMENT
MEDIUM	addons/isl/src/dag/render.ts	639	// ╰───────┤ ├─╯ │	COMMENT
MEDIUM	thrift/lib/rust/channel_pipeline/src/handler.rs	1107	// ── Phase 5: dedicated atomics, no global SEQUENCE interference ─────────────	COMMENT
MEDIUM	…rift/lib/rust/channel_pipeline/src/integration_test.rs	233	// ── Phase 6: adapter extensibility + event noop ──────────────────────	COMMENT
MEDIUM	…rift/lib/rust/channel_pipeline/src/integration_test.rs	651	// ── Phase 6: adapter extensibility + event noop ───────────────────────────	COMMENT

Verbosity Indicators76 hits · 147 pts

Severity	File	Line	Snippet	Context
LOW	…ols/admin/src/commands/derived_data/backfill_status.rs	460	// Step 1: Verify the backfill exists	COMMENT
LOW	…ols/admin/src/commands/derived_data/backfill_status.rs	489	// Step 2: Get aggregated stats	COMMENT
LOW⚡	…ols/admin/src/commands/derived_data/backfill_status.rs	503	// Step 3: Get timing stats	COMMENT
LOW⚡	…ols/admin/src/commands/derived_data/backfill_status.rs	509	// Step 4: Calculate metrics	COMMENT
LOW⚡	…rks/warm_bookmarks_cache/src/scoped_bookmarks_tests.rs	52	// Step 1: Derive only Hg data (and common dependencies), but not Git	COMMENT
LOW⚡	…rks/warm_bookmarks_cache/src/scoped_bookmarks_tests.rs	57	// Step 2: Initialize bookmarks - should include master since HgOnly is derived	COMMENT
LOW	…rks/warm_bookmarks_cache/src/scoped_bookmarks_tests.rs	87	// Step 3: Now derive Git data to make AllKinds ready	COMMENT
LOW	…rks/warm_bookmarks_cache/src/scoped_bookmarks_tests.rs	100	// Step 4: Verify bookmark now has all scopes tracked	COMMENT
LOW⚡	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	474	// Step 0: Validate if bubble deletion is enabled.	COMMENT
LOW⚡	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	478	// Step 1: Mark the bubble as expired in the backing SQL Store.	COMMENT
LOW⚡	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	492	// Step 2: Delete the blob content within the expired bubble.	COMMENT
LOW⚡	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	496	// Step 3: Delete the metadata associated with the bubble from	COMMENT
LOW	…en/mononoke/blobstore/ephemeral_blobstore/src/store.rs	525	// Step 4: Delete the bubble itself from the backing SQL store.	COMMENT
LOW	…res/commit_transformation/src/git_submodules/expand.rs	878	// Step 1: Generate the submodule metadata file change	COMMENT
LOW	…res/commit_transformation/src/git_submodules/expand.rs	928	// Step 2: Generate the deletions of files/directories that are being	COMMENT
LOW⚡	…rived_data/directory_branch_cluster_manifest/derive.rs	426	// Step 1: Load parent manifests	COMMENT
LOW⚡	…rived_data/directory_branch_cluster_manifest/derive.rs	433	// Step 2: Merge parent manifests into a single base manifest	COMMENT
LOW⚡	…rived_data/directory_branch_cluster_manifest/derive.rs	436	// Step 3: Compute cluster changes using the merged parent for lookups	COMMENT
LOW	…rived_data/directory_branch_cluster_manifest/derive.rs	450	// Step 4: Apply cluster changes to the merged parent	COMMENT
LOW	eden/mononoke/common/mononoke_configs/src/lib.rs	563	// Step 1: Separate cached from missing	COMMENT
LOW	eden/mononoke/common/mononoke_configs/src/lib.rs	583	// Step 2: Subscribe to ConfigHandles + parse OUTSIDE rcu. Accumulate	COMMENT
LOW	eden/mononoke/common/mononoke_configs/src/lib.rs	595	// Step 3: rcu — bulk insert via a single closure that re-runs on CAS	COMMENT
LOW⚡	eden/mononoke/scripts/merge_resolution_blame_checker.py	191	# Step 1: Find merge-resolved diffs	STRING
LOW	eden/mononoke/scripts/merge_resolution_blame_checker.py	212	# Step 2: Batch-check all diff IDs against trunk breakage blame	STRING
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	442	// Step 1: Build partial commit graph (this exercises merge handling)	COMMENT
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	450	// Step 2: Rewrite changesets to temporary repo	COMMENT
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	460	// Step 3: Verify the rewritten repo has correct linear history	COMMENT
LOW	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	533	// Step 1: Build partial commit graph	COMMENT
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	545	// Step 2: Rewrite changesets to temporary repo	COMMENT
LOW⚡	…/git/gitexport/tests/gitexport_tools/commit_rewrite.rs	555	// Step 3: Verify the rewritten repo has a merge commit	COMMENT
LOW	eden/scm/sapling/scmutil.py	1349	# do we need to check if the file changed?	COMMENT
LOW⚡	eden/scm/sapling/ext/snapshot/update.py	26	# Step 1: Forget files that were "hg added"	COMMENT
LOW⚡	eden/scm/sapling/ext/snapshot/update.py	33	# Step 2: Remove "untracked changes" (e.g. untracked files)	COMMENT
LOW⚡	eden/scm/sapling/ext/snapshot/update.py	42	# Step 3: Remove "tracked changes"	COMMENT
LOW	eden/scm/exec/hgmain/src/main.rs	119	// On Windows we need to check if the repository is backed by EdenFS and abort if it is not	COMMENT
LOW⚡	eden/scm/lib/indexedlog/src/log.rs	775	// Step 4: Update the indexes and folds. Optionally flush them.	COMMENT
LOW⚡	eden/scm/lib/indexedlog/src/log.rs	784	// Step 5: Write the updated meta file.	COMMENT
LOW	eden/scm/lib/indexedlog/src/log.rs	628	// Step 1: Reload metadata to get the latest view of the files.	COMMENT
LOW	eden/scm/lib/indexedlog/src/log.rs	684	// Step 2: Append to the primary log.	COMMENT
LOW	eden/scm/lib/indexedlog/src/log.rs	744	// Step 3: Reload primary log and indexes to get the latest view.	COMMENT
LOW	eden/scm/lib/manifest-tree/src/lib.rs	1261	/// The purpose of this function is to provide compatible behavior with the C++ implementation	COMMENT
LOW⚡	eden/scm/lib/dag/src/idmap/indexedlog_idmap.rs	334	// Step 1: Find (id, name) pairs in the range except for virtual items.	COMMENT
LOW⚡	eden/scm/lib/dag/src/idmap/indexedlog_idmap.rs	338	// Step 2: Write a "delete" entry to delete those indexes.	COMMENT
LOW⚡	eden/scm/lib/dag/src/idmap/indexedlog_idmap.rs	347	// Step 3: Remove entries in the virtual_map.	COMMENT
LOW	eden/scm/lib/workingcopy/src/status.rs	52	// Step 1: get the tree state for each pending change in the working copy.	COMMENT
LOW	eden/scm/lib/workingcopy/src/status.rs	180	// Step 2: handle files that aren't in pending changes.	COMMENT
LOW	eden/scm/lib/pathhistory/src/pathops.rs	232	// Step 1: Store the paths in a virtual tree.	COMMENT
LOW	eden/scm/lib/pathhistory/src/pathops.rs	251	// Step 2: BFS the tree and generate `ops`.	COMMENT
LOW⚡	eden/scm/lib/pathhistory/src/tests.rs	530	// Step 1: 1------------------------------2	COMMENT
LOW⚡	eden/scm/lib/pathhistory/src/tests.rs	531	// Step 2: 1-------------1 2	COMMENT
LOW⚡	eden/scm/lib/pathhistory/src/tests.rs	532	// Step 3: 1---3----1	COMMENT
LOW⚡	eden/scm/lib/third-party/streampager/src/direct.rs	261	// Step 1: Erase progress, and error.	COMMENT
LOW⚡	eden/scm/lib/third-party/streampager/src/direct.rs	271	// Step 2: Render new output + error + progress	COMMENT
LOW	eden/scm/lib/third-party/streampager/src/direct.rs	309	// Step 3: Update internal state.	COMMENT
LOW⚡	eden/scm/lib/cliparser/src/alias.rs	502	// Step 1: Rule: a => b $2 $1 => b y x; Result: b y x z q	COMMENT
LOW⚡	eden/scm/lib/cliparser/src/alias.rs	503	// Step 2: Rule: b => $1 c d $2 => y c d x; Result: y c d x z	COMMENT
LOW⚡	eden/scm/lib/cliparser/src/alias.rs	504	// Step 3: Rule: y => Y; Result: Y c d x z	COMMENT
LOW	eden/fs/cli_rs/edenfs-core/src/redirect.rs	2432	// The directory exist, now we need to check if there is an unknown redirection.	COMMENT
LOW⚡	eden/fs/inodes/EdenMount.cpp	2506	// Step 1: Create the callback (from public diff overload)	COMMENT
LOW⚡	eden/fs/inodes/EdenMount.cpp	2510	// Step 2: Parent validation (from full diff overload, synchronous)	COMMENT
16 more matches not shown…

Cross-File Repetition25 hits · 125 pts

Severity	File	Snippet	Context
HIGH	eden/scm/ghstack/query.py	query ($owner: string!, $name: string!, $number: int!) { repository(name: $name, owner: $owner) { id pullrequest(number:	STRING
HIGH	eden/scm/sapling/ext/github/query.py	query ($owner: string!, $name: string!, $number: int!) { repository(name: $name, owner: $owner) { id pullrequest(number:	STRING
HIGH	eden/scm/sapling/ext/github/consts/query.py	query ($owner: string!, $name: string!, $number: int!) { repository(name: $name, owner: $owner) { id pullrequest(number:	STRING
HIGH	eden/scm/ghstack/action.py	query ($owner: string!, $name: string!, $number: int!) { repository(name: $name, owner: $owner) { id pullrequest(number:	STRING
HIGH	eden/scm/ghstack/checkout.py	query ($owner: string!, $name: string!, $number: int!) { repository(name: $name, owner: $owner) { id pullrequest(number:	STRING
HIGH	eden/scm/ghstack/query.py	query ($repo_id: id!, $ref: string!) { node(id: $repo_id) { ... on repository { ref(qualifiedname: $ref) { target { oid	STRING
HIGH	eden/scm/sapling/ext/github/query.py	query ($repo_id: id!, $ref: string!) { node(id: $repo_id) { ... on repository { ref(qualifiedname: $ref) { target { oid	STRING
HIGH	eden/scm/sapling/ext/github/consts/query.py	query ($repo_id: id!, $ref: string!) { node(id: $repo_id) { ... on repository { ref(qualifiedname: $ref) { target { oid	STRING
HIGH	eden/scm/ghstack/github.py	send a get request to endpoint 'path'. returns: parsed json response	STRING
HIGH	eden/scm/ghstack/circleci.py	send a get request to endpoint 'path'. returns: parsed json response	STRING
HIGH	eden/scm/sapling/ext/github/github_endpoint.py	send a get request to endpoint 'path'. returns: parsed json response	STRING
HIGH	eden/scm/ghstack/github.py	send a post request to endpoint 'path'. returns: parsed json response	STRING
HIGH	eden/scm/ghstack/circleci.py	send a post request to endpoint 'path'. returns: parsed json response	STRING
HIGH	eden/scm/sapling/ext/github/github_endpoint.py	send a post request to endpoint 'path'. returns: parsed json response	STRING
HIGH	eden/scm/tests/github/mock_create_prs.py	setup mock github server for testing happy case of `sl pr submit` command.	STRING
HIGH	…m/tests/github/mock_create_one_pr_placeholder_issue.py	setup mock github server for testing happy case of `sl pr submit` command.	STRING
HIGH	eden/scm/tests/github/mock_create_prs_with_open.py	setup mock github server for testing happy case of `sl pr submit` command.	STRING
HIGH	eden/scm/tests/github/mock_ghrevset.py	setup mock github server for testing happy case of `sl pr submit` command.	STRING
HIGH	eden/scm/sapling/revlog.py	compare text with a given file revision returns true if text is different than what is stored.	STRING
HIGH	eden/scm/sapling/filelog.py	compare text with a given file revision returns true if text is different than what is stored.	STRING
HIGH	eden/scm/sapling/ext/remotefilelog/remotefilelog.py	compare text with a given file revision returns true if text is different than what is stored.	STRING
HIGH	…scm/sapling/ext/commitcloud/saplingremoteapiservice.py	cleanup unnecessary remote bookmarks from the given workspace	STRING
HIGH	eden/scm/sapling/ext/commitcloud/localservice.py	cleanup unnecessary remote bookmarks from the given workspace	STRING
HIGH	eden/scm/sapling/ext/commitcloud/gitservice.py	cleanup unnecessary remote bookmarks from the given workspace	STRING
HIGH	eden/scm/sapling/ext/commitcloud/baseservice.py	cleanup unnecessary remote bookmarks from the given workspace	STRING

AI Slop Vocabulary53 hits · 116 pts

Severity	File	Line	Snippet	Context
MEDIUM	eden/integration/privhelper_test.py	77	# before the harness's own cleanup) so it also covers the case	COMMENT
MEDIUM	eden/integration/privhelper_test.py	102	# Absorb that here (retry=True) instead of letting the harness's	COMMENT
LOW	eden/integration/lib/hgrepo.py	147	# If we could not find configs in the repository, just use the default	STRING
MEDIUM	eden/integration/lib/edenclient.py	322	# Merge explicit env with env vars supplied by the test harness.	COMMENT
MEDIUM	eden/integration/hg/lib/histedit_command.py	16	"""Utility to facilitate running `hg histedit` from an integration test."""	STRING
MEDIUM	eden/contrib/reviewstack/src/Link.tsx	53	// children because we need to leverage Primer to ensure :hover is honored.	COMMENT
MEDIUM	…/contrib/reviewstack/src/github/CachingGitHubClient.ts	580	// CachingGitHubClient to hang forever. As a safeguard, we leverage	COMMENT
MEDIUM	eden/mononoke/manifest/src/derive.rs	569	// Essentially performing a trivial merge between identical entries.	COMMENT
MEDIUM	eden/mononoke/tests/integration/library.sh	317	# We ask jq to count records for us, so that we're a little more robust ot	COMMENT
MEDIUM	…en/mononoke/mercurial/bundles/src/wirepack/unpacker.rs	278	// Absolutely nothing in here.	COMMENT
MEDIUM	eden/mononoke/git/git_types/src/tree.rs	492	// Essentially an empty tree. No need to store it, just create and return	COMMENT
LOW	eden/scm/gen_version.py	29	# them. It's not possible to just use double quotes instead.	COMMENT
LOW	eden/scm/tests/run-tests.py	835	# No expected output. Do not run diff but just return the (error)	COMMENT
MEDIUM	eden/scm/sapling/wireproto.py	817	# This is currently implemented mainly to facilitate testing. In most	COMMENT
LOW	eden/scm/sapling/cmdutil.py	1614	# XXX ideally, we would just pass an ctx ready to be computed, that would allow	COMMENT
LOW	eden/scm/sapling/cmdutil.py	2728	# existed in history, otherwise simply return	COMMENT
MEDIUM	eden/scm/sapling/revsetlang.py	615	r"""Quote a value in order to make it safe for the revset engine.	CODE
LOW	eden/scm/sapling/revset.py	390	# functions that just return a lot of changesets (like all) don't count here	COMMENT
LOW	eden/scm/sapling/revset.py	788	# So just return directly.	COMMENT
LOW	eden/scm/sapling/phases.py	229	# Ideally we'd just add draftrevs and publicrevs together.	COMMENT
LOW	eden/scm/sapling/context.py	1453	# Let's just return a dummy size for a submodule.	COMMENT
LOW	eden/scm/sapling/visibility.py	214	# There shouldn't be too much overlap, so we ignore this and just add	COMMENT
MEDIUM	eden/scm/sapling/repository.py	110	"""Used to facilitate debugging of arguments passed over the wire."""	STRING
LOW	eden/scm/sapling/namespaces.py	324	# if colorname is not specified, just use the logname as a backup	COMMENT
LOW	eden/scm/sapling/hgdemandimport/demandimportpy3.py	146	# PEP 302). This means that we can't just call disable + enable.	COMMENT
MEDIUM	eden/scm/sapling/ext/fbcodereview.py	696	# This makes the code more robust in case we don't learn about any	COMMENT
LOW	eden/scm/sapling/ext/sparse.py	1412	# just create a big set of include/exclude rules and	COMMENT
MEDIUM	eden/scm/sapling/ext/extlib/pywatchman/__init__.py	681	# The test harness sets WATCHMAN_BINARY to the binary under test,	COMMENT
LOW	eden/scm/sapling/ext/remotefilelog/remotefilectx.py	453	# just return whatever the linknode says for performance.	COMMENT
LOW	…n/scm/sapling/ext/remotefilelog/remotefilelogserver.py	145	# repos, so force it to just return the linkrev directly.	COMMENT
LOW	eden/scm/sapling/ext/github/pr_marker.py	46	# not a GitHubRepo, just return	COMMENT
LOW	eden/scm/sapling/ext/amend/movement.py	473	# We can't simply return heads.max() since this might give	COMMENT
MEDIUM	eden/scm/sapling/utils/cborutil.py	382	# by this semantic tag follows. A more robust parser would probably emit	COMMENT
MEDIUM	eden/scm/sapling/testing/t/runtime.py	122	# use the callback to report mismatch "in real time" and utilize	COMMENT
LOW	eden/scm/sapling/testing/t/runtime.py	454	# If TESTTMP is defined (ex. by run-tests.py), just create a	COMMENT
LOW	eden/scm/sapling/commands/debug.py	1155	# However, just return the integer value as a string so we can use	COMMENT
MEDIUM	eden/scm/lib/types/src/path.rs	21	//! guarantees. Rust has a strong type system so we can leverage it to provide more safety.	COMMENT
MEDIUM	eden/scm/lib/manifest/src/lib.rs	44	/// Fundamentally it is just a Map<file_path, file_metadata>.	COMMENT
MEDIUM	eden/scm/lib/edenapi/src/builder.rs	527	/// Configuration for a `Client`. Essentially has the same fields as a	COMMENT
MEDIUM	eden/scm/lib/http-client/src/client.rs	47	/// Essentially a more ergonomic API for working with	COMMENT
MEDIUM	eden/scm/lib/third-party/conch-parser/src/parse.rs	657	/// that are not valid file descriptors are found. Essentially used for	COMMENT
MEDIUM	eden/scm/lib/eagerepo/src/eager_repo.rs	464	// SPACE: This does not utilize zstore's delta features to save space.	COMMENT
MEDIUM	eden/fs/cli_rs/edenfs-core/src/redirect.rs	197	// metadata otherwise. We can leverage this to tell whether or not the file exists, and	COMMENT
MEDIUM	eden/fs/cli/config.py	393	"""to facilitate templatizing a centrally deployed config, we	STRING
LOW⚡	eden/fs/cli/main.py	1331	# Don't attempt to print and notify errors if we hit an exception while executing checks, just return 25	COMMENT
LOW⚡	eden/fs/cli/main.py	2918	# If we timed out waiting to become healthy, just pass	COMMENT
MEDIUM	eden/fs/cli/daemon.py	821	"""Update the EdenFS command and environment settings in order to run the	STRING
LOW	eden/fs/cli/test/lib/fake_proc_utils.py	188	pid, # inode. We just use the pid for convenience	CODE
MEDIUM	eden/fs/service/EdenServer.cpp	3376	// Alternatively, have them seamless transfer through takeovers.	COMMENT
MEDIUM	website/scripts/generate-command-markdown.py	165	# Temporary workaround: a more comprehensive escaping strategy should be	COMMENT
MEDIUM	…olWindows/InteractiveSmartlogToolWindowControl.xaml.cs	242	// Try to extract the error output for more robust matching	COMMENT
MEDIUM	addons/isl-server/src/Repository.ts	234	// Fundamentally, the intermediate states have no choice but have to	COMMENT
MEDIUM	addons/isl/src/operations/AbortMergeOperation.ts	40	// but it gets complicated. More robust is to just show a spinner on the abort button instead.	COMMENT

Redundant / Tautological Comments38 hits · 54 pts

Severity	File	Line	Snippet	Context
LOW	eden/integration/debug_getpath_test.py	141	# Close the file handle	COMMENT
LOW⚡	eden/integration/lib/journal_test_base.py	101	# Check if both have smallChange	COMMENT
LOW⚡	eden/integration/lib/journal_test_base.py	109	# Check if both have largeChange	COMMENT
LOW⚡	eden/integration/lib/journal_test_base.py	117	# Check if both have stateChange	COMMENT
LOW	eden/integration/lib/hgrepo.py	109	# Set HGRCPATH to make sure we aren't affected by the local system's	COMMENT
LOW⚡	eden/integration/hg/update_test.py	1306	# Open the file for writing, which should truncate the file	COMMENT
LOW⚡	eden/integration/hg/update_test.py	1315	# Open the file for reading, which should materialize the file	COMMENT
LOW⚡	eden/integration/hg/update_test.py	1318	# Open the file for writing, which should truncate the file	COMMENT
LOW	eden/integration/hg/update_test.py	1362	# Read file2 to hydrate its placeholder.	COMMENT
LOW	eden/scm/tests/run-tests.py	2233	exitcode = False # Set exitcode to warned.	CODE
LOW	eden/scm/sapling/merge.py	671	# Check if the file conflicts with a directory containing unknown files.	COMMENT
LOW	eden/scm/sapling/merge.py	949	# Check if all files in the conflicting directories have been removed.	COMMENT
LOW	eden/scm/sapling/bundlerepo.py	539	# Check if parents exist in localrepo before setting	COMMENT
LOW	eden/scm/sapling/ui.py	268	# Check if our repo hgrc path (or Windows UNC flavor) have already been loaded.	COMMENT
LOW	eden/scm/sapling/dagop.py	373	# Check if we need to put the working dir back in the set	COMMENT
LOW	eden/scm/sapling/dagop.py	662	# Check if we have some subgroup waiting for revisions we are not going to	COMMENT
LOW	eden/scm/sapling/windows.py	606	# Set outputencoding to UTF-8	COMMENT
LOW	eden/scm/sapling/copies.py	474	# Check if amend copytracing has been disabled.	COMMENT
LOW	eden/scm/sapling/ext/rebase.py	2466	# Check if the merge will contain unwanted changes. That may happen if	COMMENT
LOW	eden/scm/sapling/ext/progressfile.py	36	# Set pid to a fixed value for testing purpose	STRING
LOW⚡	eden/scm/sapling/ext/treemanifest/__init__.py	1358	# Check if the manifest we have is a treemanifest.	COMMENT
LOW	eden/scm/sapling/ext/amend/movement.py	207	# Check if there is an outstanding operation.	COMMENT
LOW	eden/scm/sapling/ext/fsmonitor/__init__.py	331	# Check if fsmonitor is explicitly disabled for this repository	COMMENT
LOW	eden/scm/sapling/ext/commitcloud/sync.py	281	# Check if any omissions are now included in the repo	COMMENT
LOW	eden/scm/sapling/ext/commitcloud/megarepoimport.py	176	# Check if source repo is available for transparent lookup.	COMMENT
LOW	eden/scm/sapling/testing/ext/mononoke.py	634	# Check if the 'mono' scheme is already set	STRING
LOW	eden/scm/sapling/testing/ext/mononoke.py	740	# Check if the ACL file already exists	STRING
LOW	eden/scm/sapling/testing/ext/mononoke.py	894	# Check if storageconfig is not provided and set up storage config	STRING
LOW	eden/scm/sapling/thirdparty/pysocks/socks.py	611	# Check if the destination address provided is an IP address	COMMENT
LOW	eden/fs/cli/config.py	939	# Check if it is already mounted.	COMMENT
LOW	eden/fs/cli/config.py	1385	uptime = self.get_uptime() # Check if uptime is negative?	CODE
LOW	eden/fs/cli/top.py	1066	# Check if other is more relevant	COMMENT
LOW	eden/fs/cli/debug.py	2331	# Check if any specific attributes were requested	COMMENT
LOW	eden/fs/cli/main.py	675	# Check if requested path is inside an existing checkout or backing_repo of existing checkout	COMMENT
LOW⚡	eden/fs/cli/test/health_report_test.py	115	# Open the file in write mode	COMMENT
LOW	eden/fs/cli/test/process_finder_tests.py	36	# Set counters to indicate that process 1234 has done a checkout recently.	COMMENT
LOW	eden/fs/cli/test/config_test.py	181	# Check if test is for toml or cfg by cfg._user_toml_cfg	STRING
LOW	eden/fs/cli/doctor/__init__.py	946	# Check if this checkout is nested inside another one	COMMENT

Modern Structural Boilerplate50 hits · 48 pts

Severity	File	Line	Snippet	Context
LOW	eden/integration/stats_test.py	29	logger = logging.getLogger(__name__)	CODE
LOW⚡	eden/integration/stop_test.py	260	async def update_validity_interval(self, interval: str) -> None:	CODE
LOW	eden/integration/snapshot/inode_metadata.py	121	def update_ownership(metadata_path: Path, uid: int, gid: int) -> None:	CODE
LOW	eden/integration/snapshot/snapshot.py	244	def _update_hg_state(self, base_dir: Path) -> None:	CODE
LOW	eden/integration/snapshot/snapshot.py	253	def _update_eden_state(self, base_dir: Path, uid: int, gid: int) -> None:	CODE
LOW	eden/integration/snapshot/snapshot.py	296	def _update_ownership(self, checkout_state_dir: Path, uid: int, gid: int) -> None:	CODE
LOW	eden/integration/lib/testcase.py	500	def set_rust_rollout_config(self, config: Dict[str, bool]) -> None:	CODE
LOW	eden/integration/lib/edenclient.py	988	def set_log_level(self, category: str, level: str) -> None:	CODE
LOW	eden/integration/hg/update_test.py	973	def update_to_bottom() -> None:	CODE
LOW	eden/test_support/environment_variable.py	14	def _setenv(name: str, value: Optional[str]) -> None:	CODE
LOW	eden/test_support/temporary_directory.py	209	def set_tmp_prefix(self, prefix: str) -> None:	CODE
LOW	eden/scm/sapling/dirstate.py	389	def set_xrepo_merge(self, is_xrepo_merge: bool = True) -> None:	CODE
LOW	eden/scm/sapling/statprof.py	128	__all__ = ["start", "stop", "reset", "display", "profile"]	CODE
LOW	eden/scm/sapling/lsprof.py	20	__all__ = ["profile", "Stats"]	CODE
LOW	eden/scm/sapling/hg.py	139	def _setuprepo(ui, repo, presetupfuncs=None) -> None:	CODE
LOW	eden/scm/sapling/repl_record.py	428	def _setselection(self, command: str, argument: str) -> None:	STRING
LOW	eden/scm/sapling/httpclient/__init__.py	62	logger = logging.getLogger(__name__)	CODE
LOW	eden/scm/sapling/httpclient/__init__.py	64	__all__ = ["HTTPConnection", "HTTPResponse"]	CODE
LOW	eden/scm/sapling/ext/journal.py	81	def _setupdirstate(repo, dirstate) -> None:	CODE
LOW	eden/scm/sapling/ext/sparse.py	256	def _setupupdates(_ui) -> None:	CODE
LOW	eden/scm/sapling/ext/sparse.py	445	def _setupcommit(ui) -> None:	CODE
LOW	eden/scm/sapling/ext/sparse.py	477	def _setuplog(ui) -> None:	CODE
LOW	eden/scm/sapling/ext/sparse.py	503	def _setupcat(ui) -> None:	CODE
LOW	eden/scm/sapling/ext/sparse.py	621	def _setupclone(ui) -> None:	CODE
LOW	eden/scm/sapling/ext/sparse.py	629	def _setupadd(ui) -> None:	CODE
LOW	eden/scm/sapling/ext/sparse.py	652	def _setupdirstate(ui) -> None:	CODE
LOW	eden/scm/sapling/ext/sparse.py	765	def _setupdiff(ui) -> None:	CODE
LOW	eden/scm/sapling/agent/__init__.py	8	__all__ = ["get_agent_instructions"]	CODE
LOW	eden/scm/sapling/testing/single.py	40	logger = logging.getLogger(__name__)	STRING
LOW	eden/scm/sapling/testing/t/hghave.py	22	logger = logging.getLogger(__name__)	CODE
LOW	eden/scm/sapling/testing/sh/__init__.py	603	__all__ = ["stdlib", "sheval", "Env", "Scope"]	STRING
LOW	eden/scm/sapling/thirdparty/attr/validators.py	10	__all__ = ["and_", "in_", "instance_of", "optional", "provides"]	CODE
LOW	eden/scm/sapling/thirdparty/attr/__init__.py	34	__all__ = [	CODE
LOW	eden/scm/sapling/thirdparty/attr/_config.py	4	__all__ = ["set_run_validators", "get_run_validators"]	CODE
LOW	eden/scm/i18n/polib.py	36	__all__ = [	CODE
LOW	eden/fs/py/eden/thrift/legacy.py	20	__all__ = ["EdenClient", "EdenNotRunningError", "SOCKET_PATH", "create_thrift_client"]	CODE
LOW⚡	eden/fs/cli/config.py	1027	def _set_intentionally_unmounted(self, path: str) -> None:	CODE
LOW⚡	eden/fs/cli/config.py	1033	def set_intentionally_unmounted_for_all_mounts(self) -> None:	CODE
LOW	eden/fs/cli/config.py	1486	def set_config(self, config: CheckoutConfig) -> None:	CODE
LOW	eden/fs/cli/top.py	234	def _update_screen_size(self) -> None:	CODE
LOW	eden/fs/cli/top.py	532	def _update_summary_stats(self, client) -> None:	CODE
LOW	eden/fs/cli/top.py	1090	def set_fetches(self, fetch_counts: int) -> None:	CODE
LOW	eden/fs/cli/mp.py	57	def _setup_library_paths() -> None:	CODE
LOW	eden/fs/cli/debug.py	320	def set_fetches(self, fetch_counts: int) -> None:	CODE
LOW	eden/fs/cli/hg_util.py	133	def _setup_off_mount_hg_dir(checkout_hg_dir: Path, real_hg_dir: Path) -> None:	STRING
LOW	eden/fs/cli/main.py	124	def set_sample_rate(self, automation: int) -> None:	CODE
LOW	eden/fs/cli/test/lib/fake_proc_utils.py	164	def set_file_contents(self, path: Union[Path, str], contents: bytes) -> None:	CODE
LOW	eden/fs/cli/test/lib/fake_proc_utils.py	167	def set_file_exception(self, path: Union[Path, str], exception: Exception) -> None:	CODE
LOW	eden/fs/cli/doctor/check_rogue_edenfs.py	110	def set_manual_remediation_message(self) -> None:	CODE
LOW	eden/fs/cli/doctor/test/lib/fake_client.py	95	def set_counter_value(self, counter: str, value: int) -> None:	CODE

AI Structural Patterns58 hits · 46 pts

Severity	File	Line	Context
LOW	eden/integration/glob_test.py	64	CODE
LOW	eden/integration/glob_test.py	82	CODE
LOW	eden/integration/glob_test.py	143	CODE
LOW	eden/integration/glob_test.py	191	CODE
LOW	eden/integration/projfs_buffer.py	49	CODE
LOW	eden/integration/projfs_buffer.py	129	CODE
LOW	eden/integration/lib/hgrepo.py	187	CODE
LOW	eden/integration/lib/edenclient.py	123	CODE
LOW	…ricted_path/test_infra/python/break_loader/platform.py	10	CODE
LOW	…ricted_path/test_infra/python/break_loader/platform.py	16	CODE
LOW	…ock_restricted_path/test_infra/python/simple/simple.py	9	CODE
LOW	…ock_restricted_path/test_infra/python/simple/simple.py	15	CODE
LOW	…ock_restricted_path/test_infra/python/simple/simple.py	20	CODE
LOW	…ock_restricted_path/test_infra/python/simple/simple.py	25	CODE
LOW	eden/scm/ghstack/submit.py	123	CODE
LOW	eden/scm/tests/run-tests.py	1066	CODE
LOW	eden/scm/tests/run-tests.py	2733	CODE
LOW	eden/scm/sapling/patch.py	2590	CODE
LOW	eden/scm/sapling/merge.py	2534	CODE
LOW	eden/scm/sapling/changegroup.py	442	CODE
LOW	eden/scm/sapling/util.py	4482	CODE
LOW	eden/scm/sapling/wireproto.py	1261	CODE
LOW	eden/scm/sapling/wireproto.py	491	CODE
LOW	eden/scm/sapling/cmdutil.py	1140	CODE
LOW	eden/scm/sapling/cmdutil.py	3358	CODE
LOW	eden/scm/sapling/transaction.py	150	CODE
LOW	eden/scm/sapling/localrepo.py	961	CODE
LOW	eden/scm/sapling/localrepo.py	2726	CODE
LOW	eden/scm/sapling/parser.py	237	CODE
LOW	eden/scm/sapling/crecord.py	950	CODE
LOW	eden/scm/sapling/context.py	2974	CODE
LOW	eden/scm/sapling/context.py	3318	CODE
LOW	eden/scm/sapling/lock.py	193	CODE
LOW	eden/scm/sapling/templater.py	757	CODE
LOW	eden/scm/sapling/registrar.py	179	CODE
LOW	eden/scm/sapling/namespaces.py	282	CODE
LOW	eden/scm/sapling/match.py	38	CODE
LOW	eden/scm/sapling/httpclient/__init__.py	426	CODE
LOW	eden/scm/sapling/ext/fbcodereview.py	924	CODE
LOW	eden/scm/sapling/ext/sparse.py	2861	CODE
LOW	eden/scm/sapling/ext/extlib/phabricator/graphql.py	731	CODE
LOW	eden/scm/sapling/ext/extlib/pywatchman/__init__.py	929	CODE
LOW	eden/scm/sapling/ext/commitcloud/syncstate.py	86	CODE
LOW	…scm/sapling/ext/commitcloud/saplingremoteapiservice.py	80	CODE
LOW	eden/scm/sapling/ext/commitcloud/localservice.py	105	CODE
LOW	eden/scm/sapling/ext/commitcloud/commands.py	1364	CODE
LOW	eden/scm/sapling/testing/sh/stdlib.py	226	CODE
LOW	eden/scm/sapling/commands/__init__.py	945	CODE
LOW	eden/scm/sapling/commands/__init__.py	6281	CODE
LOW	eden/scm/sapling/thirdparty/pysocks/socks.py	153	CODE
LOW	eden/scm/sapling/thirdparty/attr/_make.py	56	CODE
LOW	eden/scm/sapling/thirdparty/attr/_make.py	238	CODE
LOW	…ides/201904-segmented-changelog/segmented-changelog.py	124	CODE
LOW	eden/fs/cli/util.py	493	CODE
LOW	eden/fs/cli/util.py	513	CODE
LOW	eden/fs/cli/doctor/__init__.py	137	CODE
LOW	eden/fs/cli/doctor/test/lib/fake_eden_instance.py	108	CODE
LOW	eden/fs/cli/doctor/test/lib/fake_client.py	142	CODE

Synthetic Comment Markers5 hits · 35 pts

Severity	File	Line	Snippet	Context
HIGH	eden/contrib/reviewstack/.llms/rules/reviewstack.md	33	├── generated/ # Auto-generated code (GraphQL types, TextMate manifests)	CODE
HIGH	eden/mononoke/repo_attributes/filestore/src/lib.rs	387	/// Bytes as requested unless the file is shorter than that.	COMMENT
HIGH	eden/scm/sapling/ext/rebase.py	2358	# For non-merge changeset, just move p to adjusted dest as requested.	COMMENT
HIGH	eden/scm/lib/dag/src/tests/test_sparse.rs	697	// Can resolve A using H~3, not B~1 or D~1 as requested.	COMMENT
HIGH	addons/isl/src/stackEdit/commitStackState.ts	570	// "goto" or "reset" as requested.	COMMENT

Magic Placeholder Names4 hits · 30 pts

Severity	File	Line	Snippet	Context
HIGH⚡	…/mononoke/docs/gitimport_from_bundle_pipeline_guide.md	80	manifold --apikey <YOUR_API_KEY> put /tmp/my_repo.bundle <BUCKET_NAME>/<KEY_PATH>	CODE
HIGH⚡	…/mononoke/docs/gitimport_from_bundle_pipeline_guide.md	80	manifold --apikey <YOUR_API_KEY> put /tmp/my_repo.bundle <BUCKET_NAME>/<KEY_PATH>	CODE
HIGH⚡	…/mononoke/docs/gitimport_from_bundle_pipeline_guide.md	98	- `<YOUR_API_KEY>` — The Manifold API key for the bucket (e.g., `upstream_repo_bundles-key`). Ask your team if you don't	CODE
HIGH⚡	…/mononoke/docs/gitimport_from_bundle_pipeline_guide.md	98	- `<YOUR_API_KEY>` — The Manifold API key for the bucket (e.g., `upstream_repo_bundles-key`). Ask your team if you don't	CODE

Fake / Example Data21 hits · 24 pts

Severity	File	Line	Snippet	Context
LOW	eden/integration/lib/gitrepo.py	57	env={'GIT_AUTHOR_NAME': 'John Doe'})	STRING
LOW⚡	eden/mononoke/git/git_types/src/derive_commit.rs	427	"John Doe <john.doe@gmail.com>" => ("John Doe", "john.doe@gmail.com"),	CODE
LOW⚡	eden/mononoke/git/git_types/src/derive_commit.rs	428	"John Doe<john.doe@gmail.com>" => ("John Doe", "john.doe@gmail.com"),	CODE
LOW⚡	eden/mononoke/git/git_types/src/derive_commit.rs	429	"John Doe\t<john.doe@gmail.com>" => ("John Doe", "john.doe@gmail.com"),	CODE
LOW⚡	eden/mononoke/git/git_types/src/derive_commit.rs	431	"John Doe" => ("John Doe", ""),	CODE
LOW	eden/scm/tests/git.sh	8	export GIT_AUTHOR_EMAIL="test@example.org"	CODE
LOW	eden/scm/tests/git.sh	11	export GIT_COMMITTER_EMAIL="test@example.org"	CODE
LOW	eden/scm/tests/test-context.py	163	repo, status, text=i, user="test@test.com", date=(0, 0)	CODE
LOW	eden/scm/sapling/ext/edensparse.py	56	lorem ipsum = Keys and values are separated by a : or =	STRING
LOW	eden/scm/sapling/ext/sparse.py	63	lorem ipsum = Keys and values are separated by a : or =	STRING
LOW	eden/scm/lib/virtual-repo/src/text_gen.rs	16	const LOREM_IPSUM: &str = r#"Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do	CODE
LOW	eden/scm/lib/virtual-repo/src/text_gen.rs	16	const LOREM_IPSUM: &str = r#"Lorem ipsum dolor sit amet, consectetur adipiscing elit, sed do	CODE
LOW	eden/fs/inodes/test/RenameTest.cpp	858	{"a/b/testdir/sample.txt", "Lorem ipsum dolor sit amet\n"},	CODE
LOW	eden/fs/inodes/test/RenameTest.cpp	858	{"a/b/testdir/sample.txt", "Lorem ipsum dolor sit amet\n"},	CODE
LOW	eden/fs/inodes/overlay/WindowsFsck.cpp	110	// - A directory can be "placeholder" even if one of it's recursive descendants	COMMENT
LOW	addons/components/explorer/ComponentExplorer.tsx	296	<TextArea placeholder="placeholder" onChange={e => console.log(e.currentTarget.value)}>	CODE
LOW	addons/components/explorer/ComponentExplorer.tsx	299	<TextField placeholder="placeholder" onChange={e => console.log(e.currentTarget.value)}>	CODE
LOW⚡	thrift/lib/rust/any/src/dummy_any.rs	25	fn get_dummy_any(dummy_data: &str) -> Self;	CODE
LOW⚡	thrift/lib/rust/any/src/dummy_any.rs	30	fn get_dummy_any(dummy_data: &str) -> any::Any {	CODE
LOW⚡	thrift/lib/rust/any/src/dummy_any.rs	33	name: TypeName::structType(TypeUri::uri(dummy_data.to_string())),	CODE
LOW⚡	thrift/lib/rust/any/src/dummy_any.rs	37	data: dummy_data.into(),	CODE

Slop Phrases15 hits · 22 pts

Severity	File	Line	Snippet	Context
LOW	eden/contrib/shared/Comparison.ts	83	* Note: non-localized. Don't forget to run this through `t()` for a given client.	COMMENT
MEDIUM	eden/mononoke/repo_attributes/filestore/src/prepare.rs	132	// NOTE: The Multiplexer makes clones of the Bytes we pass in. It's worth noting that Bytes	COMMENT
LOW	eden/mononoke/mononoke_types/src/derivable_type.rs	233	// If the compiler reminds you to add something here, please don't forget to also	COMMENT
LOW	eden/scm/ghstack/logs.py	125	# If you ever switch this, make sure to preserve redaction	COMMENT
LOW	eden/scm/sapling/eden_update.py	105	# We do still need to make sure to update the merge state though.	COMMENT
LOW	eden/scm/sapling/dispatch.py	243	# make sure to check fileset first, as revset can invoke fileset	COMMENT
LOW	eden/scm/sapling/revset.py	2017	# since matching the diff is very costly, make sure to	COMMENT
MEDIUM	eden/scm/sapling/statprof.py	95	Right now the profiler is quite simplistic. I cannot provide	STRING
LOW	eden/scm/sapling/hg.py	839	repo.ui.status(_("(branch merge, don't forget to commit)\n"))	CODE
LOW	eden/scm/sapling/testing/sh/stdlib.py	646	# Clear vars out first so we don't forget to unset them.	COMMENT
LOW	eden/scm/sapling/testing/sh/stdlib.py	646	# Clear vars out first so we don't forget to unset them.	COMMENT
LOW	eden/scm/sapling/commands/subtree.py	427	repo.ui.status(_("(subtree merge, don't forget to commit)\n"))	CODE
LOW	eden/scm/sapling/commands/__init__.py	893	msg = _("changeset %s backed out, don't forget to commit.\n")	CODE
MEDIUM	website/docs/commands/commands-markdown.json	1259	"doc": "\ncopy commits from a different location\n\nUse Sapling's merge logic to copy individual commits fr	CODE
LOW	addons/shared/Comparison.ts	154	* Note: non-localized. Don't forget to run this through `t()` for a given client.	COMMENT

TODO Padding9 hits · 14 pts

Severity	File	Line	Snippet	Context
LOW	eden/scm/lib/commitcloudsubscriber/src/receiver.rs	71	// TODO: implement proper health_check request	COMMENT
LOW	eden/scm/lib/pathmatcher/src/utils.rs	14	/// TODO: Fix error handling so it returns an error when the input is	COMMENT
LOW	eden/scm/lib/edenapi/types/src/token.rs	123	// TODO: implement secure signed tokens	COMMENT
LOW	eden/fs/nfs/Nfsd3.cpp	1711	// TODO: implement on windows	COMMENT
LOW	eden/fs/inodes/OverlayFileAccess.cpp	299	// TODO: implement readFile with pread instead of lseek.	COMMENT
LOW	eden/fs/inodes/NfsDispatcherImpl.cpp	460	// TODO: implement readdirplus on Windows.	COMMENT
LOW	eden/fs/inodes/NfsDispatcherImpl.cpp	485	// TODO: implement statfs on windows	COMMENT
LOW	eden/fs/rust/edenfs-asserted-states-client/src/lib.rs	79	// TODO: Add logging	COMMENT
LOW	addons/isl/src/stackEdit/ui/EditStackModal.tsx	37	// TODO: implement absorb model.	COMMENT

Hallucination Indicators1 hit · 10 pts

Severity	File	Line	Snippet	Context
CRITICAL	…s/slapi/slapi_server/repo_listener/src/http_service.rs	477	match HeaderValue::from_str(this.conn.pending.acceptor.server_hostname.as_str())	CODE

Docstring Block Structure2 hits · 10 pts

Severity	File	Line	Snippet	Context
HIGH	eden/fs/py/eden/thrift/client.py	42	Create a synchronous Thrift client for communicating with the Eden Thrift server. We default to no per-RPC tim	STRING
HIGH	eden/fs/py/eden/thrift/client.py	96	Create a Thrift client for communicating with the Eden Thrift server. We default to no per-RPC timeout because	STRING

Dead Code3 hits · 6 pts

Severity	File	Line	Context
MEDIUM	eden/scm/tests/failfilemerge.py	12	CODE
MEDIUM	eden/scm/sapling/eden_dirstate.py	58	CODE
MEDIUM	eden/fs/cli/test/remove_legacyephemeral_test.py	306	CODE

Cross-Language Confusion (JS/TS)1 hit · 5 pts

Severity	File	Line	Snippet	Context
HIGH	…contrib/reviewstack/src/textmate/createGrammarStore.ts	53	print(str: string): void {	CODE

Overly Generic Function Names4 hits · 4 pts

Severity	File	Line	Snippet	Context
LOW	eden/scm/sapling/ext/fbcodereview.py	619	def helper(*args):	CODE
LOW	eden/scm/sapling/ext/github/pr_status.py	72	def helper(repo, *pr_list: PullRequestId) -> List[Optional[GraphQLPullRequest]]:	CODE
LOW	addons/shared/__tests__/stringify.test.ts	491	export function doSomething() {	CODE
LOW	addons/shared/__tests__/stringify.test.ts	517	export function doSomething() {	CODE

Modern AI Meta-Vocabulary1 hit · 3 pts

Severity	File	Line	Snippet	Context
MEDIUM	…mononoke/repo_attributes/restricted_paths/test/main.rs	1047	// `client_identity_regexes`. The agentic caller's id is still	COMMENT

Example Usage Blocks1 hit · 2 pts

Severity	File	Line	Snippet	Context
LOW	eden/scm/sapling/ext/sampling.py	8	# Usage:	COMMENT

Maintenance and Governance Review

Risk Review · v3

What These Metrics Mean

Pattern-Signal History

Severity Breakdown

Directory Score Breakdown

Pattern Findings