AI Slop Code Risk Monitor

vitessio/vitess

SlopCodeMonitor badge for vitessio/vitess Adjusted legacy signal; badge bands: 25 / 50 / 75 (different from dashboard bands).

Vitess is a database clustering system for horizontal scaling of MySQL.

0.9 Low pattern signal View on GitHub

Open-Source Code Quality, Security and Technical Debt Review

This report summarizes scanner evidence for vitessio/vitess, a Go project with 21,397 GitHub stars. SynthScan examined 1,511,689 lines of code across 4658 source files, recording 1108 pattern matches distributed across 17 syntactic categories. The legacy adjusted pattern signal is 0.9 (Low pattern signal); it is not an authorship estimate.

Use the findings to ask whether maintainers can explain intent, verify dependencies, simplify unnecessary scaffolding, and focus review effort. Risk Payload v1 adds ownership, supply-chain, over-engineering, and review-effort vectors when available. These are prompts for human verification, not probabilities, security findings, or measured maintenance costs.

0.9
Adjusted Signal
0.9
Raw Pattern Score
100%
Time Factor
2026-10-31
Last Push
21.4K
Stars
Go
Language
1.5M
Lines of Code
4.7K
Files
1.1K
Pattern Hits
2026-10-31
Scan Date
0.00
HC Hit Rate

Risk Review · Payload v1

Use each vector to guide manual verification of the underlying findings.

Payload schema: 1.0.0 · Engine: 2.0.0 · Scoring: heuristic-density-v1 · Scan date: 2026-10-31.

Cognitive Ownership Risk 3.9/100 · LOW

Can maintainers explain intent, assumptions, and trade-offs? 1073 finding(s).

Hallucinated Dependency & Supply Chain Risk 0.5/100 · LOW

Are imports declared and package/API names verified? 30 finding(s).

Agent Over-Engineering Ratio 0.1/100 · LOW

Are wrappers, guards, and repeated structures necessary? 15 finding(s).

PR Review Velocity Impact 4.3/100 · LOW

Which changes need focused human review? 1108 finding(s).

What These Metrics Mean

Adjusted Signal
Legacy pattern score normalized per 1,000 scanned lines and multiplied by the repository-level time factor. It is not an estimate of AI authorship.
Raw Pattern Score
Sum of severity- and context-weighted pattern findings before applying the time factor.
Time Factor
A coarse multiplier based on repository last-push metadata. It does not identify when individual code was written.
Pattern Hits
Count of configured pattern findings. Review each location and surrounding code before acting.
HC Hit Rate
High- and critical-severity findings per scanned file. Severity is a configured detector weight, not proof of a vulnerability or authorship.
Lines / Files
Scan coverage denominators used by the legacy normalized score.

Pattern-Signal History

This chart tracks the legacy adjusted pattern score across scan runs. Changes can reflect source changes, scan coverage, or detector/scoring versions; they do not measure AI adoption or prove that a team refactored code. Compare runs only when their scoring versions are compatible. The dashed line tracks the raw pattern-hit count.

Severity Breakdown

Groups findings by configured severity weight. Severity is not a confidence estimate, security rating, or proof that a package is missing. Verify imports against the project's manifests and inspect each finding in its local design context.

CRITICAL 15HIGH 1MEDIUM 45LOW 1047

Directory Score Breakdown

This horizontal bar chart shows where configured pattern scores accumulate across directories. Treat the bars as locations to inspect, not as evidence of authorship or code quality. Use the file-level findings to decide whether a directory needs a focused ownership, dependency, complexity, or review check.

Pattern Findings

The scanner identified 1108 distinct pattern matches across 17 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 Block953 hits · 953 pts
SeverityFileLineSnippetContext
LOWbootstrap.sh1#!/bin/bashCOMMENT
LOWmisc/parse_cover.py1#!/usr/bin/pythonCOMMENT
LOWtools/e2e_test_cluster.sh1#!/bin/bashCOMMENT
LOWtools/all_test_for_coverage.sh1#!/bin/bashCOMMENT
LOWtools/get_previous_release.sh1#!/bin/bashCOMMENT
LOWtools/build_version_flags.sh1#!/bin/bashCOMMENT
LOWtools/e2e_test_race.sh1#!/bin/bashCOMMENT
LOWtools/e2e_test_race.sh21COMMENT
LOWtools/unit_test_runner.sh1#!/bin/bashCOMMENT
LOWtools/release_utils.sh1#!/bin/bashCOMMENT
LOWtools/code_freeze.sh1#!/bin/bashCOMMENT
LOWtools/code_freeze.sh21# The first argument of the script is a string, which should be either: "freeze" or "unfreeze".COMMENT
LOWtools/warm_go_build_cache.sh1#!/bin/bashCOMMENT
LOWtools/warm_go_build_cache.sh21#COMMENT
LOWtools/get_next_release.sh1#!/bin/bashCOMMENT
LOWtools/pylint.sh1#!/bin/bashCOMMENT
LOWtools/assert_tests_ran.sh1#!/bin/bashCOMMENT
LOWtools/remove_dependencies.sh1#!/bin/bashCOMMENT
LOWtools/create_release.sh1#!/bin/bashCOMMENT
LOWtools/dependency_check.sh1#!/bin/bashCOMMENT
LOWtools/e2e_go_test.sh21COMMENT
LOWtools/check_license_headers.sh1#!/bin/bashCOMMENT
LOWtools/back_to_dev_mode.sh1#!/bin/bashCOMMENT
LOWtools/e2e_test_runner.sh1#!/bin/bashCOMMENT
LOWtools/e2e_test_runner.sh21COMMENT
LOWtools/delete_superseded_cache_entries.sh1#!/bin/bashCOMMENT
LOWdocker/bootstrap/build.sh1#!/bin/bashCOMMENT
LOWdocker/test/run.sh1#!/bin/bashCOMMENT
LOWdocker/test/run.sh21# internally use this script to launch the test for you.COMMENT
LOWdocker/vttestserver/run.sh1#!/bin/bashCOMMENT
LOWdocker/vttestserver/setup_vschema_folder.sh1#!/bin/bashCOMMENT
LOWdocker/mini/vtctld-mini-up.sh1#!/bin/bashCOMMENT
LOWdocker/mini/vttablet-mini-up.sh1#!/bin/bashCOMMENT
LOWdocker/binaries/logrotate/rotate.sh1#!/bin/bashCOMMENT
LOWdocker/binaries/logtail/tail.sh1#!/bin/bashCOMMENT
LOWgo/vt/vtgate/vstream_manager.go181COMMENT
LOWgo/vt/vtgate/scatter_conn.go721}COMMENT
LOWgo/vt/vtgate/temp_table_refresh.go41// idle timer and mysqld's wait_timeout clock. That is intentional: theCOMMENT
LOWgo/vt/vtgate/temp_table_refresh.go61// session, so their count is client-controlled; without a cap a hostileCOMMENT
LOWgo/vt/vtgate/temp_table_refresh.go81 // parity for both the MySQL protocol and the gRPC API, complementing theCOMMENT
LOWgo/vt/vtgate/temp_table_refresh.go121COMMENT
LOWgo/vt/vtgate/temp_table_refresh.go161// (a rare race sending one extra refresh is harmless). Shard sessions with anCOMMENT
LOWgo/vt/vtgate/executor_dml_test.go3061}COMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go121}COMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go161 return tempTableTargetKey{alias: topoproto.TabletAliasString(t.alias), reservedID: t.reservedID}COMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go201 vtg *VTGateCOMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go301 // A session-level transaction timeout that does not exceed the worst-caseCOMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go401 }()COMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go501// correct — the tablet is unreachable.COMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go521// beat rather than accumulating one every tick. And a tablet whose last beatCOMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go561 // taken right away, so a tablet that has recovered — its beatCOMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go661 }COMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go741// connections (false on a tablet predating the keepalive).COMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go841// ACL and query-log path, so the identity is not used there; it matters on aCOMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go861// user traffic and reclaims idle sessions exactly as MySQL would. On a tabletCOMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go941 // temp tables dropped, session and user variables reset, LAST_INSERT_IDCOMMENT
LOWgo/vt/vtgate/plugin_mysql_server.go1481 return vterrors.VT12001("ComRegisterReplica for the VTGate handler")COMMENT
LOWgo/vt/vtgate/plan_execute.go201 if err != nil { // The error we're checking here is the return value from the newExecute functionCOMMENT
LOWgo/vt/vtgate/executorcontext/safe_session.go101 // being set. Otherwise, it starts as notAutocommitable.COMMENT
LOWgo/vt/vtgate/executorcontext/safe_session.go481// FindAndChangeSessionIfInSingleTxMode retrieves the ShardSession matching the given keyspace, shard, and tablet type.COMMENT
893 more matches not shown…
Hallucination Indicators15 hits · 150 pts
SeverityFileLineSnippetContext
CRITICALgo/vt/vtgate/plan_execute.go103 if e.resolver.scatterConn.gateway.buffer != nil && e.resolver.scatterConn.gateway.buffer.GetConfig() != nil {CODE
CRITICALgo/vt/vtgate/plan_execute.go104 timeout = e.resolver.scatterConn.gateway.buffer.GetConfig().MaxFailoverDuration / (MaxBufferingRetries - 1)CODE
CRITICALgo/vt/vttablet/tabletmanager/vdiff/table_differ.go1073 statement, err := td.wd.ct.vde.parser.Parse(td.tablePlan.sourceQuery)CODE
CRITICALgo/vt/vttablet/tabletmanager/vdiff/table_plan.go90 statement, err := td.wd.ct.vde.parser.Parse(td.sourceQuery)CODE
CRITICALgo/vt/vttablet/tabletmanager/vdiff/report.go90 statement, err := td.wd.ct.vde.parser.Parse(queryStmt)CODE
CRITICALgo/vt/vttablet/tabletserver/tabletserver.go2385 if err := tsv.te.txPool.scp.conns.SetCapacity(ctx, int64(val)); err != nil {CODE
CRITICALgo/vt/vttablet/tabletserver/tabletserver.go2388 return tsv.te.txPool.scp.foundRowsPool.SetCapacity(ctx, int64(val))CODE
CRITICALgo/vt/vttablet/tabletserver/tabletserver_test.go181 require.EqualValues(t, 1, tsv.te.txPool.scp.active.Size(), "tsv.te.txPool.scp.active.Size()")CODE
CRITICALgo/vt/vttablet/tabletserver/tabletserver_test.go249 tsv.te.txPool.scp.lastID.Store(1)CODE
CRITICALgo/vt/vttablet/tabletserver/tabletserver_test.go296 assert.EqualValues(c, 20, tsv.te.txPool.scp.lastID.Load(), "tsv.te.txPool.lastID.Get()")CODE
CRITICALgo/vt/vttablet/tabletserver/tabletserver_test.go2547 assert.EqualValues(t, 1, tsv.te.txPool.scp.tempTableUnmanaged.Load(), "the marked, unmanaged connection must be on the CODE
CRITICALgo/vt/vttablet/tabletserver/tabletserver_test.go2564 assert.Zero(t, tsv.te.txPool.scp.tempTableUnmanaged.Load(), "a keepalive-managed connection must leave the gauge")CODE
CRITICALgo/vt/vttablet/tabletserver/tabletserver_test.go2648 assert.Zero(t, tsv.te.txPool.scp.tempTableUnmanaged.Load(), "a drop-only connection must stay off the gauge")CODE
CRITICALgo/vt/vttablet/tabletserver/query_executor_test.go3643 return tsv.te.txPool.scp.conns.Metrics.DiscardedByCallerCount()CODE
CRITICALgo/vt/vttablet/tabletserver/query_executor.go1559 tabletTypeStr := qre.tsv.sm.target.TabletType.String()CODE
AI Slop Vocabulary21 hits · 57 pts
SeverityFileLineSnippetContext
MEDIUMgo/vt/vtgate/planbuilder/operators/queryprojection.go604// We are also free to add more ORDER BY columns than the user asked for which we leverage,COMMENT
MEDIUM…/vt/vttablet/tabletmanager/vreplication/vreplicator.go201// All phases share the same plan building framework. We leverage the fact theCOMMENT
MEDIUMgo/vt/vttablet/tabletserver/tabletserver.go94// Essentially, InitDBConfig is a continuation of New. However,COMMENT
MEDIUMgo/vt/vttablet/tabletserver/vstreamer/rowstreamer.go211 // We wish to utilize a UNIQUE KEY which is not the PRIMARY KEY/COMMENT
MEDIUMgo/vt/vttablet/tabletserver/vstreamer/rowstreamer.go282 // leverage the ordering from the index itself and avoid having toCOMMENT
MEDIUM…vt/vttablet/tabletserver/txserializer/tx_serializer.go363// transactions. Instead, we leverage that Go routines waiting for a channelCOMMENT
MEDIUMgo/vt/grpcoptionaltls/tls_detector.go35 // Definitely not TLSCOMMENT
MEDIUMgo/vt/vtorc/logic/topology_recovery.go1050 // is not dead, then we will proceed with the fix for the replica. Essentially, we are trading off speed in replica recCOMMENT
MEDIUMgo/cmd/vtctl/vtctl.go59 // Essentially, the situation we have here is that commands look like:COMMENT
MEDIUMgo/flags/endtoend/analyze_flags.py45 """Analyze all binary help files and generate comprehensive report."""STRING
MEDIUMgo/flags/endtoend/analyze_flags.py92 """Generate comprehensive migration report."""STRING
MEDIUMgo/test/endtoend/cluster/vttablet_process.go588// This allows the tests to be more robust in the face of transient failures.COMMENT
MEDIUMgo/test/endtoend/cluster/vttablet_process.go625// This allows the tests to be more robust in the face of transient failures.COMMENT
MEDIUMgo/test/endtoend/vreplication/vdiff_helper_test.go39 vdiffTimeout = 180 * time.Second // We can leverage auto retry on error with this longer-than-usual timeoutCODE
MEDIUMgo/test/endtoend/reparent/utils/utils.go564 // it's possible that we get other ephemeral errors too, so we make the tests more robust byCOMMENT
MEDIUM…/tabletmanager/disk_health_monitor/fuse_helper/main.go141// shouldn't happen with a stock LoopbackRoot, but keeps the helper robust ifCOMMENT
MEDIUM…/endtoend/vtorc/primaryfailure/primary_failure_test.go486 // log format. We should change this to something more robust.COMMENT
MEDIUMweb/vtadmin/package-lock.json11361 "resolved": "https://registry.npmjs.org/robust-predicates/-/robust-predicates-3.0.2.tgz",CODE
MEDIUMweb/vtadmin/tests/server.ts35// means our fake is more robust than it would be otherwise. Since we are usingCOMMENT
MEDIUMweb/vtadmin/src/hooks/useSyncedURLParam.ts62 // TODO(doeg): One possible, more nuanced re-implementation is to push entries onto the stackCOMMENT
MEDIUMweb/vtadmin/src/hooks/useTheme.ts13// of the Debug page. The :) correct implementation :) will be more robust,COMMENT
Self-Referential Comments19 hits · 56 pts
SeverityFileLineSnippetContext
MEDIUMbootstrap.sh18### This file is executed by 'make tools'. You do not need to execute it directly.COMMENT
MEDIUMtools/create_release.sh66 ## Create the commit for this release and tag itCOMMENT
MEDIUMdocker/vttestserver/setup_vschema_folder.sh54 # Create a vschema.json file only if the number of shards are more than 1COMMENT
MEDIUMgo/vt/mysqlctl/mysqld_test.go465 assert.Contains(t, template, "# This file is auto-included when MySQL 9.0 or later is detected.")CODE
MEDIUMgo/vt/mysqlctl/mysqld_test.go473 assert.Contains(t, template, "# This file is auto-included when MySQL 9.0 or later is detected.")CODE
MEDIUM⚡…nd/vreplication/testdata/config/init_testserver_db.sql1# This file is for testing purpose only.COMMENT
MEDIUM⚡…nd/vreplication/testdata/config/init_testserver_db.sql2# This file is executed immediately after initializing a fresh data directory.COMMENT
MEDIUMconfig/init_db.sql1# This file is executed immediately after initializing a fresh data directory.COMMENT
MEDIUMexamples/compose/101_initial_cluster.sh26# Create the keyspace before starting tablets to avoid racing withCOMMENT
MEDIUMexamples/local/402_list_backup.sh19KEYSPACE="customer" # Define the keyspace to work withCODE
MEDIUMexamples/local/402_list_backup.sh20SHARDS=("-80" "80-") # Define the shards within the keyspace to list backups forCODE
MEDIUM⚡examples/local/403_restore_from_backup.sh21KEYSPACE="customer" # Define the keyspace to work withCODE
MEDIUM⚡examples/local/403_restore_from_backup.sh22SHARDS=("-80" "80-") # Define the shards within the keyspace to restoreCODE
MEDIUMexamples/local/101_initial_cluster.sh47 # Create the keyspace with the sidecar database name and set theCOMMENT
MEDIUMexamples/common/lib/utils.sh17# This file contains utility functions that can be used throughout theCOMMENT
MEDIUMexamples/operator/101_initial_cluster.yaml147 # This file is executed immediately after mysql_install_db,COMMENT
MEDIUM.github/workflows/backport.yml271 # Create the pull request with labels (returns URL like https://github.com/owner/repo/pull/123)COMMENT
MEDIUM.github/workflows/backport.yml467 # Create the pull request with labels (returns URL like https://github.com/owner/repo/pull/123)COMMENT
MEDIUMdoc/vtctl_go_reference.py480 # Create a data structure to contain information about the commandCOMMENT
Structural Annotation Overuse18 hits · 35 pts
SeverityFileLineSnippetContext
LOWgo/vt/vtgate/planbuilder/operators/union.go229 // Step 1: Identify column references in the first source's expression and record their offsets.COMMENT
LOW⚡go/vt/vtgate/planbuilder/operators/union.go251 // Step 2: Add the fully substituted expression as a column in the first source's column list.COMMENT
LOW⚡go/vt/vtgate/planbuilder/operators/union.go254 // Step 3: For each subsequent source, use the same offsets to replace column references,COMMENT
LOW⚡go/vt/vttablet/tabletmanager/vreplication/vcopier.go281// Step 1: catchup: During this step, it replicates from the source from the last position.COMMENT
LOW⚡go/vt/vttablet/tabletmanager/vreplication/vcopier.go284// Step 2: Start streaming. This returns the initial field info along with the GTIDCOMMENT
LOW⚡go/vt/vttablet/tabletmanager/vreplication/vcopier.go286// Step 3: fastForward: The target is fast-forwarded to the GTID obtained. This shouldCOMMENT
LOW⚡go/vt/vttablet/tabletmanager/vreplication/vcopier.go289// Step 4: copy rows: Copy the next set of rows from the stream that was started in Step 2.COMMENT
LOW⚡…nd/vreplication/testdata/config/init_testserver_db.sql11# WARNING: This sql is *NOT* safe for production use,COMMENT
LOW.agents/skills/resolve-backport/SKILL.md10## Step 1: Parse input and validateCOMMENT
LOW.agents/skills/resolve-backport/SKILL.md42## Step 2: Gather contextCOMMENT
LOW.agents/skills/resolve-backport/SKILL.md63## Step 3: Rebase onto latest base branchCOMMENT
LOW.agents/skills/resolve-backport/SKILL.md81## Step 4: Ensure correct Go versionCOMMENT
LOW⚡.agents/skills/resolve-backport/SKILL.md106## Step 5: Resolve conflictsCOMMENT
LOW⚡.agents/skills/resolve-backport/SKILL.md116## Step 6: Verify file scopeCOMMENT
LOW⚡.agents/skills/resolve-backport/SKILL.md122## Step 7: Run unit testsCOMMENT
LOW⚡.agents/skills/resolve-backport/SKILL.md131## Step 8: BuildCOMMENT
LOW.agents/skills/resolve-backport/SKILL.md143## Step 9: Format and lintCOMMENT
LOW.agents/skills/resolve-backport/SKILL.md168## Step 10: ShipCOMMENT
Verbosity Indicators18 hits · 32 pts
SeverityFileLineSnippetContext
LOWgo/vt/vtgate/planbuilder/operators/delete.go108 // If they aren't then we don't need the multi-table delete. But this check isn't so straight-forward. We need to checkCOMMENT
LOWgo/vt/vtgate/planbuilder/operators/join.go138 // if we are inside a CTE, we need to check if we depend on the recursion tableCOMMENT
LOWgo/vt/vtgate/planbuilder/operators/subquery_planning.go59 // but if we don't have groupings, we need to check if there are aggregations that will mess with usCOMMENT
LOWgo/vt/vtgate/planbuilder/operators/horizon_expanding.go326 // first we need to check if we have all columns or there are still unexpanded starsCOMMENT
LOWgo/vt/vtgate/planbuilder/operators/union.go229 // Step 1: Identify column references in the first source's expression and record their offsets.COMMENT
LOW⚡go/vt/vtgate/planbuilder/operators/union.go251 // Step 2: Add the fully substituted expression as a column in the first source's column list.COMMENT
LOW⚡go/vt/vtgate/planbuilder/operators/union.go254 // Step 3: For each subsequent source, use the same offsets to replace column references,COMMENT
LOWgo/vt/vttablet/tabletmanager/tm_init.go811 // tablet process was just restarted. However, we need to check if a newCOMMENT
LOW⚡go/vt/vttablet/tabletmanager/vreplication/vcopier.go281// Step 1: catchup: During this step, it replicates from the source from the last position.COMMENT
LOW⚡go/vt/vttablet/tabletmanager/vreplication/vcopier.go284// Step 2: Start streaming. This returns the initial field info along with the GTIDCOMMENT
LOW⚡go/vt/vttablet/tabletmanager/vreplication/vcopier.go286// Step 3: fastForward: The target is fast-forwarded to the GTID obtained. This shouldCOMMENT
LOW⚡go/vt/vttablet/tabletmanager/vreplication/vcopier.go289// Step 4: copy rows: Copy the next set of rows from the stream that was started in Step 2.COMMENT
LOWgo/vt/topo/wildcards.go278 // This is an expanded path, we need to check if it exists.COMMENT
LOWgo/vt/topo/zk2topo/utils.go243 // this is an expanded path, we need to check if it existsCOMMENT
LOWgo/vt/discovery/healthcheck.go552 // If the previous tablet type was primary, we need to check if the next new primary has already been assigned.COMMENT
LOWgo/vt/dbconfigs/credentials.go241 // so we need to check if they have been set by NewConfigCOMMENT
LOWgo/vt/schemadiff/schema.go1133 // Possibly dropping a foreign key; we need to check if this constraint references another table.COMMENT
LOWgo/mysql/vault/auth_server_vault.go92 // so we need to check if they have been set by NewConfigCOMMENT
Redundant / Tautological Comments13 hits · 20 pts
SeverityFileLineSnippetContext
LOWgo/flags/endtoend/analyze_flags.py22 # Check if we're entering the flags sectionCOMMENT
LOWchangelog/tooling/scripts/count-progress.sh8# Check if we're in an analysis directoryCOMMENT
LOW⚡examples/local/403_restore_from_backup.sh32 # Check if any replica tablets were foundCOMMENT
LOWexamples/common/backup-env.sh75# Set aliases to simplify instructions.COMMENT
LOWexamples/common/env.sh75# Set aliases to simplify instructions.COMMENT
LOW.github/workflows/error_code_docs.yml121 # Check if there are actual changes.COMMENT
LOW.github/workflows/error_code_docs.yml172 # Check if PR already exists.COMMENT
LOW.github/workflows/backport.yml206 # Check if there are conflictsCOMMENT
LOW.github/workflows/backport.yml402 # Check if there are conflictsCOMMENT
LOW.github/workflows/pr_opened_tasks.yml46 # Check if PR has Backport or Forwardport label (case-insensitive)COMMENT
LOW.github/workflows/arewefastyet_comment.yml42 # Check if arewefastyet comment already existsCOMMENT
LOW.github/workflows/update_golang_version.yml85 # Check if the PR already exists, if it does then do not create new PR.COMMENT
LOWdoc/vtctl_go_reference.py153 # Check if the argument name ends in a digit to catch things likeCOMMENT
Deep Nesting12 hits · 12 pts
SeverityFileLineSnippetContext
LOWgo/flags/endtoend/find_flag_definitions.py12CODE
LOWgo/flags/endtoend/find_flag_definitions.py87CODE
LOWdoc/vitess_api_reference.py29CODE
LOWdoc/vitess_api_reference.py82CODE
LOWdoc/vitess_api_reference.py136CODE
LOWdoc/vitess_api_reference.py244CODE
LOWdoc/vitess_api_reference.py362CODE
LOWdoc/vitess_api_reference.py380CODE
LOWdoc/vitess_api_reference.py403CODE
LOWdoc/vitess_api_reference.py524CODE
LOWdoc/vitess_api_reference.py572CODE
LOWdoc/vitess_api_reference.py604CODE
Fake / Example Data11 hits · 11 pts
SeverityFileLineSnippetContext
LOWgo/vt/vtgate/testdata/executor_vexplain.json41 "query": "select u.foo, ue.bar, count(*) from user u join user_extra ue on u.id = ue.user_id where u.name = 'John DoCODE
LOWgo/vt/vtgate/testdata/executor_vexplain.json102 "query": "update user set name = 'Jane Doe' where id = 1",CODE
LOWgo/vt/vtgate/testdata/executor_vexplain.json120 "query": "replace into user (id, name) values (1, 'Jane Doe')",CODE
LOWgo/vt/schemadiff/schema_test.go1113 query := strings.ReplaceAll(tableBase, "placeholder", tableName)CODE
LOWgo/vt/schemadiff/schema_test.go1117 query = strings.ReplaceAll(modifiedTable, "placeholder", tableName)CODE
LOW…nd/onlineddl/vrepl_suite/testdata/json57dml/create.sql23 insert into onlineddl_test (id, i, j) values (null, 19, '{"text":"Lorem ipsum dolor sit amet, consectetuer adipiscing CODE
LOW…nd/onlineddl/vrepl_suite/testdata/json57dml/create.sql23 insert into onlineddl_test (id, i, j) values (null, 19, '{"text":"Lorem ipsum dolor sit amet, consectetuer adipiscing CODE
LOWweb/vtadmin/src/components/routes/Settings.tsx109 Lorem ipsum dolor sit amet, consectetur adipiscing elit. Morbi in facilisis libero,CODE
LOWweb/vtadmin/src/components/routes/Settings.tsx109 Lorem ipsum dolor sit amet, consectetur adipiscing elit. Morbi in facilisis libero,CODE
LOWweb/vtadmin/src/components/routes/Settings.tsx139 sodales, mauris est iaculis dui, ut tristique nibh sapien ac sapien. Lorem ipsumCODE
LOWweb/vtadmin/src/components/routes/Settings.tsx140 dolor sit amet, consectetur adipiscing elit.CODE
Hyper-Verbose Identifiers10 hits · 10 pts
SeverityFileLineSnippetContext
LOWgo/vt/servenv/status.go110function refreshTablesHaveClassRefreshRequired() {CODE
LOWtest/vtop_example.sh50function checkPodStatusWithTimeout() {CODE
LOWtest/vtop_example.sh83function waitForKeyspaceToBeServing() {CODE
LOWdoc/vtctl_go_reference.py340def get_group_name_from_variable(file_path, variable_name):CODE
LOWdoc/vitess_api_reference.py69def print_method_summary_group_row(doc, group_name):CODE
LOWdoc/vitess_api_reference.py208def print_message_detail_header(doc, proto, message_details, message_name,CODE
LOWdoc/vitess_api_reference.py227def print_method_detail_header(doc, method):CODE
LOWdoc/vitess_api_reference.py380def print_method_detail_request(doc, proto_contents, proto, method):CODE
LOWdoc/vitess_api_reference.py403def print_method_detail_response(doc, proto_contents, proto, method):CODE
LOWdoc/vitess_api_reference.py426def print_proto_file_definition(doc, proto_contents, proto):CODE
Excessive Try-Catch Wrapping4 hits · 8 pts
SeverityFileLineSnippetContext
MEDIUMgo/flags/endtoend/find_flag_definitions.py18 print("Error: VTROOT environment variable is not set. Please set it to the Vitess root directory.")CODE
MEDIUMgo/flags/endtoend/find_flag_definitions.py91 print("Error: VTROOT environment variable is not set. Please set it to the Vitess root directory.")CODE
MEDIUMgo/flags/endtoend/analyze_flags.py48 print("Error: VTROOT environment variable is not set. Please set it to the Vitess root directory.")CODE
MEDIUMgo/flags/endtoend/analyze_flags.py95 print("Error: VTROOT environment variable is not set. Please set it to the Vitess root directory.")CODE
Example Usage Blocks4 hits · 6 pts
SeverityFileLineSnippetContext
LOWtools/delete_superseded_cache_entries.sh20# Usage:COMMENT
LOWdocker/bootstrap/build.sh17# Usage:COMMENT
LOWgo/cmd/internal/docgen/docgen.go20// Example usage:COMMENT
LOWgo/mysql/icuregex/internal/ucase/fold.go66# Usage:COMMENT
Empty Error Handlers1 hit · 5 pts
SeverityFileLineSnippetContext
HIGHgo/flags/endtoend/find_flag_definitions.py60except:CODE
TODO Padding3 hits · 4 pts
SeverityFileLineSnippetContext
LOWgo/mysql/icuregex/matcher.go915 // TODO: implement transparent bounds. Ticket #6067COMMENT
LOWgo/mysql/icuregex/matcher.go1267 // TODO: implementCOMMENT
LOWgo/mysql/icuregex/matcher.go1351 // TODO: implementCOMMENT
Unused Imports4 hits · 4 pts
SeverityFileLineSnippetContext
LOWgo/flags/endtoend/find_flag_definitions.py7CODE
LOWgo/flags/endtoend/find_flag_definitions.py9CODE
LOWdoc/vitess_api_reference.py20CODE
LOWdoc/vitess_api_reference.py23CODE
Modern AI Meta-Vocabulary1 hit · 3 pts
SeverityFileLineSnippetContext
MEDIUM…vttablet/tabletmanager/vreplication/controller_plan.go77// provide guardrails for data changing statements and if the user wants getCOMMENT
Slop Phrases1 hit · 1 pts
SeverityFileLineSnippetContext
LOWtools/code_freeze.sh65echo "(make sure to replace upstream with vitessio/vitess' remote)"CODE