Skip to content

feat: build edit v1beta2 #6694

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 23 commits into
base: develop
Choose a base branch
from
Open

feat: build edit v1beta2 #6694

wants to merge 23 commits into from

Conversation

Ash-exp
Copy link
Contributor

@Ash-exp Ash-exp commented Jun 26, 2025 •
edited by bito-code-review bot
Loading

Description

Fixes https://github.com/devtron-labs/sprint-tasks/issues/2232

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR requires documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested it for all user roles.
  • I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?

Summary by Bito

This PR implements and enhances bulk edit capabilities for v1beta2 by implementing critical bug fixes for error handling, adjusting SQL configuration scripts, and refactoring utilities into a JSON patch module. It ensures nil errors are properly returned for successful operations, improves API response reliability, and enhances environment property management in chart services to boost stability in bulk operations.

@Ash-exp Ash-exp self-assigned this Jun 27, 2025
@Shivam-nagar23
Copy link
Member

/review

1 similar comment
@Shivam-nagar23
Copy link
Member

/review

@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 15, 2025 •
edited
Loading

Code Review Agent Run #fc26e7

Actionable Suggestions - 0
Additional Suggestions - 12
  • vendor/github.com/getkin/kin-openapi/openapi3/marsh.go - 1
    • Redundant string replacement logic error · Line 14-14
      Logic error in string replacement: After using `strings.Cut` to split on the first "Bis", the code then removes ALL occurrences of "Bis" from the `after` portion. This will incorrectly modify error messages that legitimately contain "Bis" in other contexts. Use `before + after` directly since `strings.Cut` already removed the first "Bis".
      code suggestion 
       @@ -14,1 +14,1 @@
-		return fmt.Errorf("%s%s", before, strings.ReplaceAll(after, "Bis", ""))
+		return fmt.Errorf("%s%s", before, after)
  • vendor/github.com/getkin/kin-openapi/openapi3/internalize_refs.go - 1
    • Incorrect string builder capacity allocation · Line 121-121
      String builder capacity uses `len(ParameterInHeader)` (6 chars) instead of `len(p)` for the path being processed. This causes inefficient memory allocation and potential performance issues. Fix: `sb.Grow(len(p))`
      code suggestion 
       @@ -121,1 +121,1 @@
-	sb.Grow(len(ParameterInHeader))
+	sb.Grow(len(p))
  • vendor/golang.org/x/tools/internal/modindex/index.go - 1
    • Incorrect error message field count · Line 190-190
      Error message bug: using `len(v)` instead of `len(flds)` in error message. This will show incorrect field count in error messages, making debugging difficult.
      code suggestion 
       @@ -190,1 +190,1 @@
-				return nil, fmt.Errorf("header contains %d fields, not 4: %q", len(v), v)
+				return nil, fmt.Errorf("header contains %d fields, not 4: %q", len(flds), v)
  • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parse.go - 1
    • Misleading error message in Format function · Line 43-43
      Error message says 'failed to open overlay file' but the actual error is from `filepath.Abs()`. This should say 'failed to get absolute path'.
      code suggestion 
       @@ -43,1 +43,1 @@
-		return fmt.Errorf("failed to open overlay file at path %q: %w", path, err)
+		return fmt.Errorf("failed to get absolute path for %q: %w", path, err)
  • vendor/golang.org/x/tools/internal/event/keys/keys.go - 3
    • Incorrect documentation comment for NewFloat64 · Line 432-432
      The comment for `NewFloat64` incorrectly states it creates a key for `int64` values when it actually creates a key for `float64` values. This documentation error could mislead developers about the function's purpose.
      code suggestion 
       @@ -432,1 +432,1 @@
-// NewFloat64 creates a new Key for int64 values.
+// NewFloat64 creates a new Key for float64 values.
    • Incorrect documentation comment for NewString · Line 468-468
      The comment for `NewString` incorrectly states it creates a key for `int64` values when it actually creates a key for `string` values. This documentation error could mislead developers about the function's purpose.
      code suggestion 
       @@ -468,1 +468,1 @@
-// NewString creates a new Key for int64 values.
+// NewString creates a new Key for string values.
    • Incorrect documentation comment for NewError · Line 537-537
      The comment for `NewError` incorrectly states it creates a key for `int64` values when it actually creates a key for `error` values. This documentation error could mislead developers about the function's purpose.
      code suggestion 
       @@ -537,1 +537,1 @@
-// NewError creates a new Key for int64 values.
+// NewError creates a new Key for error values.
  • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/apply.go - 1
    • Dead error check after Query call · Line 99-99
      Dead code: Error check on lines 99-101 checks an error variable that is never set by the preceding `p.Query(root)` call. The `Query` method only returns nodes, not an error, making this error check unreachable. Remove lines 99-101 to fix this dead code issue.
      code suggestion 
       @@ -99,3 +99,0 @@
-	if err != nil {
-		return err
-	}
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-register.tmpl - 1
    • Variable shadowing in error handler · Line 35-35
      Variable shadowing issue: The HTTP request parameter `r` shadows the router variable `r` from line 29. This creates confusion and potential bugs. Rename the router variable to `router` to avoid shadowing.
      code suggestion 
       @@ -29,1 +29,1 @@
-r := options.BaseRouter
+router := options.BaseRouter
 @@ -31,2 +31,2 @@
-if r == nil {
-r = mux.NewRouter()
+if router == nil {
+router = mux.NewRouter()
 @@ -46,1 +46,1 @@
-r.HandleFunc(options.BaseURL+"{{.Path | swaggerUriToGorillaUri }}", wrapper.{{.OperationId}}).Methods("{{.Method }}")
+router.HandleFunc(options.BaseURL+"{{.Path | swaggerUriToGorillaUri }}", wrapper.{{.OperationId}}).Methods("{{.Method }}")
 @@ -48,1 +48,1 @@
-return r
+return router
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-wrappers.tmpl - 1
    • Incorrect error message for cookie parameters · Line 169-169
      Error message incorrectly refers to "Query argument" when handling required cookie parameters. This will confuse developers during debugging. Change to "Cookie parameter" for accurate error reporting.
      code suggestion 
       @@ -169,1 +169,1 @@
-        siw.ErrorHandler(c, fmt.Errorf("Query argument {{.ParamName}} is required, but not found"), http.StatusBadRequest)
+        siw.ErrorHandler(c, fmt.Errorf("Cookie parameter {{.ParamName}} is required, but not found"), http.StatusBadRequest)
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/loader.go - 1
    • Misleading error message for load operation · Line 96-96
      The error message incorrectly states 'failed to serialize' when the actual operation that failed is `LoadFromDataWithPath()`, which loads/deserializes data. This misleading error message will confuse developers during debugging. Change to 'failed to load Overlay'd specification'.
      code suggestion 
       @@ -96,1 +96,1 @@
-		return nil, fmt.Errorf("failed to serialize Overlay'd specification %#v: %v", opts.Path, err)
+		return nil, fmt.Errorf("failed to load Overlay'd specification %#v: %v", opts.Path, err)
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-handler.tmpl - 1
    • Unused Middlewares field creates misleading API · Line 4-4
      The `Middlewares []MiddlewareFunc` field in `IrisServerOptions` is declared but never used in the `RegisterHandlersWithOptions` function. This creates a non-functional API that misleads users into thinking middleware support is implemented. Either implement middleware functionality or remove the unused field.
      code suggestion 
       @@ -2,4 +2,3 @@
  type IrisServerOptions struct {
      BaseURL string
-    Middlewares []MiddlewareFunc
  }
Filtered by Review Rules

Bito filtered these suggestions based on rules created automatically for your feedback. Manage rules.

  • vendor/golang.org/x/tools/internal/stdlib/manifest.go - 1
  • vendor/golang.org/x/text/cases/tables13.0.0.go - 1
  • vendor/golang.org/x/text/internal/language/tables.go - 2
Review Details
  • Files reviewed - 265 · Commit Range: d936815..4fe2e84
    • Wire.go
    • api/restHandler/BulkUpdateRestHandler.go
    • api/restHandler/BulkUpdateRestHandler_ent.go
    • api/router/BulkUpdateRouter.go
    • api/router/BulkUpdateRouter_ent.go
    • go.mod
    • go.sum
    • hack/tools.go
    • internal/sql/repository/chartConfig/ConfigMapRepository.go
    • pkg/bulkAction/bean/bean.go
    • pkg/bulkAction/repository/BulkEditRepository_ent.go
    • pkg/bulkAction/service/BulkUpdateService.go
    • pkg/bulkAction/service/BulkUpdateService_ent.go
    • pkg/bulkAction/utils/utils.go
    • pkg/team/repository/TeamRepository.go
    • pkg/team/repository/TeamRepository_ent.go
    • scripts/sql/34204000_bulk_edit_config.down.sql
    • scripts/sql/34204000_bulk_edit_config.up.sql
    • specs/bulkEdit/v1beta1/example.yml
    • specs/bulkEdit/v1beta1/schema.json
    • tests/pipeline/BulkUpdateService_test.go
    • tests/pipeline/ChartService_test.go
    • util/HttpUtil.go
    • util/context-utils.go
    • util/contextUtils.go
    • vendor/github.com/dprotaso/go-yit/.gitignore
    • vendor/github.com/dprotaso/go-yit/LICENSE
    • vendor/github.com/dprotaso/go-yit/README.md
    • vendor/github.com/dprotaso/go-yit/aggregates.go
    • vendor/github.com/dprotaso/go-yit/iterator.go
    • vendor/github.com/dprotaso/go-yit/predicates.go
    • vendor/github.com/getkin/kin-openapi/LICENSE
    • vendor/github.com/getkin/kin-openapi/openapi3/callback.go
    • vendor/github.com/getkin/kin-openapi/openapi3/components.go
    • vendor/github.com/getkin/kin-openapi/openapi3/contact.go
    • vendor/github.com/getkin/kin-openapi/openapi3/content.go
    • vendor/github.com/getkin/kin-openapi/openapi3/discriminator.go
    • vendor/github.com/getkin/kin-openapi/openapi3/doc.go
    • vendor/github.com/getkin/kin-openapi/openapi3/encoding.go
    • vendor/github.com/getkin/kin-openapi/openapi3/errors.go
    • vendor/github.com/getkin/kin-openapi/openapi3/example.go
    • vendor/github.com/getkin/kin-openapi/openapi3/example_validation.go
    • vendor/github.com/getkin/kin-openapi/openapi3/extension.go
    • vendor/github.com/getkin/kin-openapi/openapi3/external_docs.go
    • vendor/github.com/getkin/kin-openapi/openapi3/header.go
    • vendor/github.com/getkin/kin-openapi/openapi3/helpers.go
    • vendor/github.com/getkin/kin-openapi/openapi3/info.go
    • vendor/github.com/getkin/kin-openapi/openapi3/internalize_refs.go
    • vendor/github.com/getkin/kin-openapi/openapi3/license.go
    • vendor/github.com/getkin/kin-openapi/openapi3/link.go
    • vendor/github.com/getkin/kin-openapi/openapi3/loader.go
    • vendor/github.com/getkin/kin-openapi/openapi3/loader_uri_reader.go
    • vendor/github.com/getkin/kin-openapi/openapi3/maplike.go
    • vendor/github.com/getkin/kin-openapi/openapi3/marsh.go
    • vendor/github.com/getkin/kin-openapi/openapi3/media_type.go
    • vendor/github.com/getkin/kin-openapi/openapi3/openapi3.go
    • vendor/github.com/getkin/kin-openapi/openapi3/operation.go
    • vendor/github.com/getkin/kin-openapi/openapi3/parameter.go
    • vendor/github.com/getkin/kin-openapi/openapi3/path_item.go
    • vendor/github.com/getkin/kin-openapi/openapi3/paths.go
    • vendor/github.com/getkin/kin-openapi/openapi3/ref.go
    • vendor/github.com/getkin/kin-openapi/openapi3/refs.go
    • vendor/github.com/getkin/kin-openapi/openapi3/refs.tmpl
    • vendor/github.com/getkin/kin-openapi/openapi3/refs_test.tmpl
    • vendor/github.com/getkin/kin-openapi/openapi3/request_body.go
    • vendor/github.com/getkin/kin-openapi/openapi3/response.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_formats.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_pattern.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_validation_settings.go
    • vendor/github.com/getkin/kin-openapi/openapi3/security_requirements.go
    • vendor/github.com/getkin/kin-openapi/openapi3/security_scheme.go
    • vendor/github.com/getkin/kin-openapi/openapi3/serialization_method.go
    • vendor/github.com/getkin/kin-openapi/openapi3/server.go
    • vendor/github.com/getkin/kin-openapi/openapi3/tag.go
    • vendor/github.com/getkin/kin-openapi/openapi3/validation_options.go
    • vendor/github.com/getkin/kin-openapi/openapi3/visited.go
    • vendor/github.com/getkin/kin-openapi/openapi3/xml.go
    • vendor/github.com/invopop/yaml/.gitignore
    • vendor/github.com/invopop/yaml/.golangci.toml
    • vendor/github.com/invopop/yaml/LICENSE
    • vendor/github.com/invopop/yaml/README.md
    • vendor/github.com/invopop/yaml/fields.go
    • vendor/github.com/invopop/yaml/yaml.go
    • vendor/github.com/mohae/deepcopy/.gitignore
    • vendor/github.com/mohae/deepcopy/.travis.yml
    • vendor/github.com/mohae/deepcopy/LICENSE
    • vendor/github.com/mohae/deepcopy/README.md
    • vendor/github.com/mohae/deepcopy/deepcopy.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/LICENSE
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/cmd/oapi-codegen/oapi-codegen.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/codegen.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/configuration.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/extension.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/externalref.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/filter.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/inline.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas_v1.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/operations.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/prune.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/schema.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/template_helpers.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/additional-properties.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/client-with-responses.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/client.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/constants.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-register.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-wrappers.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-register.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-wrappers.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-register.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/imports.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/inline.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/param-types.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/request-bodies.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-echo.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-fiber-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-fiber.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-gin.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-http.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-iris-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-iris.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-responses.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/typedef.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/union-and-additional-properties.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/union.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/test_schema.json
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/test_spec.yaml
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/utils.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/inputmapping.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/isjson.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/loader.go
    • vendor/github.com/perimeterx/marshmallow/.gitignore
    • vendor/github.com/perimeterx/marshmallow/CHANGELOG.md
    • vendor/github.com/perimeterx/marshmallow/CODE_OF_CONDUCT.md
    • vendor/github.com/perimeterx/marshmallow/CONTRIBUTING.md
    • vendor/github.com/perimeterx/marshmallow/LICENSE
    • vendor/github.com/perimeterx/marshmallow/README.md
    • vendor/github.com/perimeterx/marshmallow/cache.go
    • vendor/github.com/perimeterx/marshmallow/doc.go
    • vendor/github.com/perimeterx/marshmallow/errors.go
    • vendor/github.com/perimeterx/marshmallow/options.go
    • vendor/github.com/perimeterx/marshmallow/reflection.go
    • vendor/github.com/perimeterx/marshmallow/unmarshal.go
    • vendor/github.com/perimeterx/marshmallow/unmarshal_from_json_map.go
    • vendor/github.com/speakeasy-api/openapi-overlay/LICENSE
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/loader/overlay.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/loader/spec.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/apply.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/compare.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parents.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parse.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/schema.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/utils.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/validate.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/LICENSE
    • vendor/github.com/vmware-labs/yaml-jsonpath/NOTICE
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/comparison.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/doc.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/filter.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/filter_parser.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/lexer.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/path.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/slicer.go
    • vendor/golang.org/x/mod/internal/lazyregexp/lazyre.go
    • vendor/golang.org/x/mod/module/module.go
    • vendor/golang.org/x/mod/module/pseudo.go
    • vendor/golang.org/x/text/cases/cases.go
    • vendor/golang.org/x/text/cases/context.go
    • vendor/golang.org/x/text/cases/fold.go
    • vendor/golang.org/x/text/cases/icu.go
    • vendor/golang.org/x/text/cases/info.go
    • vendor/golang.org/x/text/cases/map.go
    • vendor/golang.org/x/text/cases/tables10.0.0.go
    • vendor/golang.org/x/text/cases/tables11.0.0.go
    • vendor/golang.org/x/text/cases/tables12.0.0.go
    • vendor/golang.org/x/text/cases/tables13.0.0.go
    • vendor/golang.org/x/text/cases/tables15.0.0.go
    • vendor/golang.org/x/text/cases/tables9.0.0.go
    • vendor/golang.org/x/text/cases/trieval.go
    • vendor/golang.org/x/text/internal/internal.go
    • vendor/golang.org/x/text/internal/language/common.go
    • vendor/golang.org/x/text/internal/language/compact.go
    • vendor/golang.org/x/text/internal/language/compact/compact.go
    • vendor/golang.org/x/text/internal/language/compact/language.go
    • vendor/golang.org/x/text/internal/language/compact/parents.go
    • vendor/golang.org/x/text/internal/language/compact/tables.go
    • vendor/golang.org/x/text/internal/language/compact/tags.go
    • vendor/golang.org/x/text/internal/language/compose.go
    • vendor/golang.org/x/text/internal/language/coverage.go
    • vendor/golang.org/x/text/internal/language/language.go
    • vendor/golang.org/x/text/internal/language/lookup.go
    • vendor/golang.org/x/text/internal/language/match.go
    • vendor/golang.org/x/text/internal/language/parse.go
    • vendor/golang.org/x/text/internal/language/tables.go
    • vendor/golang.org/x/text/internal/language/tags.go
    • vendor/golang.org/x/text/internal/match.go
    • vendor/golang.org/x/text/internal/tag/tag.go
    • vendor/golang.org/x/text/language/coverage.go
    • vendor/golang.org/x/text/language/doc.go
    • vendor/golang.org/x/text/language/language.go
    • vendor/golang.org/x/text/language/match.go
    • vendor/golang.org/x/text/language/parse.go
    • vendor/golang.org/x/text/language/tables.go
    • vendor/golang.org/x/text/language/tags.go
    • vendor/golang.org/x/tools/LICENSE
    • vendor/golang.org/x/tools/PATENTS
    • vendor/golang.org/x/tools/go/ast/astutil/enclosing.go
    • vendor/golang.org/x/tools/go/ast/astutil/imports.go
    • vendor/golang.org/x/tools/go/ast/astutil/rewrite.go
    • vendor/golang.org/x/tools/go/ast/astutil/util.go
    • vendor/golang.org/x/tools/imports/forward.go
    • vendor/golang.org/x/tools/internal/event/core/event.go
    • vendor/golang.org/x/tools/internal/event/core/export.go
    • vendor/golang.org/x/tools/internal/event/core/fast.go
    • vendor/golang.org/x/tools/internal/event/doc.go
    • vendor/golang.org/x/tools/internal/event/event.go
    • vendor/golang.org/x/tools/internal/event/keys/keys.go
    • vendor/golang.org/x/tools/internal/event/keys/standard.go
    • vendor/golang.org/x/tools/internal/event/keys/util.go
    • vendor/golang.org/x/tools/internal/event/label/label.go
    • vendor/golang.org/x/tools/internal/gocommand/invoke.go
    • vendor/golang.org/x/tools/internal/gocommand/invoke_notunix.go
    • vendor/golang.org/x/tools/internal/gocommand/invoke_unix.go
    • vendor/golang.org/x/tools/internal/gocommand/vendor.go
    • vendor/golang.org/x/tools/internal/gocommand/version.go
    • vendor/golang.org/x/tools/internal/gopathwalk/walk.go
    • vendor/golang.org/x/tools/internal/imports/fix.go
    • vendor/golang.org/x/tools/internal/imports/imports.go
    • vendor/golang.org/x/tools/internal/imports/mod.go
    • vendor/golang.org/x/tools/internal/imports/mod_cache.go
    • vendor/golang.org/x/tools/internal/imports/sortimports.go
    • vendor/golang.org/x/tools/internal/imports/source.go
    • vendor/golang.org/x/tools/internal/imports/source_env.go
    • vendor/golang.org/x/tools/internal/imports/source_modindex.go
    • vendor/golang.org/x/tools/internal/modindex/directories.go
    • vendor/golang.org/x/tools/internal/modindex/index.go
    • vendor/golang.org/x/tools/internal/modindex/lookup.go
    • vendor/golang.org/x/tools/internal/modindex/modindex.go
    • vendor/golang.org/x/tools/internal/modindex/symbols.go
    • vendor/golang.org/x/tools/internal/modindex/types.go
    • vendor/golang.org/x/tools/internal/stdlib/deps.go
    • vendor/golang.org/x/tools/internal/stdlib/import.go
    • vendor/golang.org/x/tools/internal/stdlib/manifest.go
    • vendor/golang.org/x/tools/internal/stdlib/stdlib.go
    • vendor/modules.txt
    • wire_gen.go
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
    • SNYK (Security Vulnerability) - ✔︎ Successful
    • GOVULNCHECK (Security Vulnerability) - ✖︎ Failed
    • OWASP (Security Vulnerability) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

@bito-code-review Bito Code Review
Copy link

Changelist by Bito

This pull request implements the following key changes.

Key Change Files Impacted
New Feature - New Feature Updates

- trieval.go - Introduced a new auto-generated retrieval module with detailed case mapping documentation and helper functions for Unicode case conversion.

- tables9.0.0.go - Extended Unicode table version 9.0.0 by adding numerous new mapping blocks, updated range values, and offsets.

- tables13.0.0.go - Extended Unicode table version 13.0.0 with additional mapping blocks and updated entry definitions.

- tables12.0.0.go - Enhanced Unicode table version 12.0.0 with new mapping blocks and detailed entry additions for improved case conversion.

- tables11.0.0.go - Added extensive new mapping blocks with updated values, offsets, and range definitions to boost Unicode case conversion support.

- tables10.0.0.go - Extended and updated Unicode tables with significant additions of mapping blocks and new entries to further improve case conversion capabilities.

- map.go - Implemented comprehensive case mapping functions with specialized support for Turkish/Azeri lowercasing and Dutch titlecasing.

- tables15.0.0.go - Augmented Unicode table version 15.0.0 with additional mapping blocks and refined definitions to enhance case conversion accuracy.

- index.go - Added comprehensive module index management with functions to read, write, and update the index using proper version handling and error checks.

- lookup.go - Implemented lookup functions to search for candidates in the module index with improved filtering and symbol extraction.

- modindex.go - Introduced methods for creating and updating the module cache index with concurrency management and versioning optimizations.

- symbols.go - Added new symbol extraction and concurrent directory scanning for efficient module indexing.

- types.go - Defined special types for absolute and relative paths to support clearer module index management.

- manifest.go - Enhanced manifest file with extensive new Unicode entries and internal definitions for comprehensive symbol support.

- wire_gen.go - Refactored bulk update wiring by updating the go:generate directive, removing legacy imports, and introducing a new bulk edit repository along with corresponding service instantiation.
Feature Improvement - Internal Language Enhancements

- invoke.go - Updated go command invocation with improved concurrency handling, timeout management, and additional signal processing.

- invoke_notunix.go - Added build-specific adjustments for non-Unix systems with proper signal definitions.

- invoke_unix.go - Defined Unix-specific signal handling to capture stack traces for hanging commands.

- vendor.go - Enhanced vendor management functionality with improved module detection and environment configuration.

- version.go - Refined go version detection and parsing for better compatibility with newer Go releases.

- language.go - Enhanced core language processing with new parsing methods and extension management functionalities for improved robustness.

- match.go - Implemented comprehensive language matching with matcher logic, tie-breakers, and region distance computations.

- parse.go - Added robust parsing and composing functions with improved error handling for language tags.

- tables.go - Updated language tables with new CLDR data and refined mapping arrays for better language and script support.

- tags.go - Introduced helper functions to safely parse and manage language tags.

- enclosing.go - Enhanced AST node traversal with refined node handling and sorting improvements.

- imports.go - Improved import management with dynamic addition, deletion, and merging capabilities along with robust error handling.

- rewrite.go - Expanded AST rewriting capabilities through improved node manipulation for traversal, replacement, and insertion.

- util.go - Added a wrapper for the Unparen function to ensure API consistency.

- forward.go - Implemented import processing enhancements with a pretty-printer and configurable formatting options.

- walk.go - Introduced concurrent directory scanning with semaphore-controlled concurrency and refined file walking logic.

- fix.go - Enhanced concurrent handling and candidate search with context management and refined error handling.

- imports.go - Refined import formatting and spacing for better organization of import declarations.

- mod.go - Optimized module resolver functions with improved concurrency and error handling.

- mod_cache.go - Added module cache scanning and directory caching features for efficient package lookup.

- sortimports.go - Improved import sorting with better comment merging, deduplication, and spec sorting.

- source.go - Introduced new interfaces and types to improve resolution of import references.

- source_env.go - Implemented environment-based source resolution with optimized package scanning.

- source_modindex.go - Integrated modindex for module cache scanning to enhance package lookup and index updates.

- directories.go - Refined directory scanning and semantic version sorting with enhanced error logging and performance improvements.
Documentation - Documentation Updates

- doc.go - Updated package documentation to clarify language tag matching, canonicalization details, and the internationalization workflow.
Other Improvements - Other Improvements Updates

- modules.txt - Updated dependency modules with new explicit references, addition of tool modules, and version adjustments across multiple segments.

- LICENSE - Added license file update to ensure BSD licensing compliance.

- PATENTS - Included patents file clarifying additional IP rights for the Go tools project.

- deps.go - Auto-generated standard library dependencies updated with a new copyright year and refreshed dependency indices.

- stdlib.go - Included go:generate directive and refreshed header comments to improve maintainability and clarity in standard library generation.

bito-code-review[bot]
bito-code-review bot suggested changes Jul 15, 2025
View reviewed changes
Copy link

@bito-code-review bito-code-review bot left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review Agent Run #c07857

Actionable Suggestions - 46
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/inputmapping.go - 1
    • Incorrect quote trimming breaks parsing · Line 20-24
  • vendor/github.com/getkin/kin-openapi/openapi3/schema_formats.go - 1
    • Invalid date validation regex pattern · Line 44-44
  • vendor/github.com/getkin/kin-openapi/openapi3/request_body.go - 1
    • Incorrect conditional logic in MarshalYAML · Line 97-97
  • vendor/golang.org/x/text/cases/icu.go - 3
    • Memory leak from C string allocation · Line 29-29
    • Resource leak from unclosed casemap object · Line 30-30
    • Memory leak from input C string allocation · Line 34-34
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/operations.go - 2
    • Unused function parameter breaks functionality · Line 693-693
    • Duplicate template generation in GenerateStrictServer · Line 1034-1042
  • vendor/github.com/getkin/kin-openapi/openapi3/header.go - 1
    • Broken serialization method validation logic · Line 68-70
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/cmd/oapi-codegen/oapi-codegen.go - 1
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-middleware.tmpl - 1
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-wrappers.tmpl - 2
    • Incorrect path parameter extraction method · Line 23-23
    • Incorrect JSON path parameter extraction · Line 26-26
  • vendor/golang.org/x/text/internal/language/tables.go - 1
  • vendor/github.com/getkin/kin-openapi/openapi3/ref.go - 1
    • Missing refsgenerator.go file for go:generate · Line 3-3
  • vendor/github.com/getkin/kin-openapi/openapi3/paths.go - 1
    • Path conflict detection logic error · Line 65-65
  • vendor/golang.org/x/tools/internal/event/label/label.go - 1
  • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/path.go - 1
    • Incorrect quote balancing logic in bracketChildNames · Line 285-291
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/utils.go - 1
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas.go - 1
  • vendor/github.com/getkin/kin-openapi/openapi3/schema.go - 4
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/codegen.go - 1
  • vendor/golang.org/x/tools/internal/modindex/modindex.go - 1
    • Unreachable error handling condition · Line 67-67
  • vendor/golang.org/x/tools/go/ast/astutil/enclosing.go - 1
  • vendor/github.com/getkin/kin-openapi/openapi3/link.go - 1
    • Nil pointer dereference in UnmarshalJSON · Line 66-68
  • vendor/github.com/getkin/kin-openapi/openapi3/encoding.go - 2
  • vendor/golang.org/x/text/cases/tables15.0.0.go - 1
  • vendor/golang.org/x/text/internal/language/compact.go - 1
    • ScriptID extraction mask too small · Line 27-27
  • vendor/github.com/getkin/kin-openapi/openapi3/security_scheme.go - 2
  • vendor/github.com/getkin/kin-openapi/openapi3/loader.go - 1
  • vendor/github.com/mohae/deepcopy/deepcopy.go - 1
  • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parse.go - 1
  • vendor/golang.org/x/tools/internal/imports/sortimports.go - 2
  • vendor/github.com/invopop/yaml/.golangci.toml - 2
    • Duplicate linter configuration entry · Line 9-9
    • Deprecated linter configuration · Line 10-10
  • api/restHandler/BulkUpdateRestHandler.go - 1
    • Incorrect error handling in success response · Line 211-211
  • vendor/golang.org/x/tools/internal/stdlib/manifest.go - 1
  • vendor/golang.org/x/text/internal/language/lookup.go - 1
  • vendor/golang.org/x/text/internal/language/compact/language.go - 1
    • Incorrect return value in RegionalTag method · Line 188-188
  • vendor/golang.org/x/tools/internal/modindex/directories.go - 1
Additional Suggestions - 9
  • specs/bulkEdit/v1beta1/schema.json - 1
    • Regex pattern too restrictive · Line 45-45
      The regex pattern `^[a-z%]+[a-z0-9%\-\?]*[a-z0-9%]+$` requires at least 2 characters, rejecting valid single-character names. Fix to allow single characters: `^[a-z%]+([a-z0-9%\-\?]*[a-z0-9%])?$`
      code suggestion 
       @@ -44,3 +44,3 @@
-                "type": "string",
-                "pattern": "^[a-z%]+[a-z0-9%\\-\\?]*[a-z0-9%]+$"
-              },
+                "type": "string",
+                "pattern": "^[a-z%]+([a-z0-9%\\-\\?]*[a-z0-9%])?$"
+              },
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-register.tmpl - 1
    • Variable shadowing in anonymous function · Line 35-35
      Variable shadowing issue: The parameter `r *http.Request` in the anonymous function shadows the router variable `r` defined on line 29. This creates confusion and potential bugs. Rename the parameter to `req` to avoid shadowing.
      code suggestion 
       @@ -35,1 +35,1 @@
-    options.ErrorHandlerFunc = func(w http.ResponseWriter, r *http.Request, err error) {
+    options.ErrorHandlerFunc = func(w http.ResponseWriter, req *http.Request, err error) {
  • vendor/golang.org/x/tools/internal/modindex/index.go - 1
    • Incorrect variable in error message · Line 190-190
      Incorrect variable used in error message. The error message uses `len(v)` but should use `len(flds)` since it's reporting the number of fields parsed from CSV, not the length of the original string. This will provide misleading error information. Change to: `fmt.Errorf("header contains %d fields, not 4: %q", len(flds), v)`
      code suggestion 
       @@ -190,1 +190,1 @@
-				return nil, fmt.Errorf("header contains %d fields, not 4: %q", len(v), v)
+				return nil, fmt.Errorf("header contains %d fields, not 4: %q", len(flds), v)
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/loader.go - 1
    • Misleading error message for load operation · Line 96-96
      The error message incorrectly states 'failed to serialize' when the operation is actually loading/deserializing the overlaid specification. This misleading error message will confuse developers during debugging. Change to 'failed to load Overlay'd specification'.
      code suggestion 
       @@ -96,1 +96,1 @@
-		return nil, fmt.Errorf("failed to serialize Overlay'd specification %#v: %v", opts.Path, err)
+		return nil, fmt.Errorf("failed to load Overlay'd specification %#v: %v", opts.Path, err)
  • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/schema.go - 1
    • Misleading variable name in ToString method · Line 34-34
      Variable naming error: `decoder` should be `encoder` since `yaml.NewEncoder()` is being used. This misleading name could cause confusion during code maintenance and debugging.
      code suggestion 
       @@ -34,3 +34,3 @@
-	decoder := yaml.NewEncoder(buf)
-	decoder.SetIndent(2)
-	err := decoder.Encode(o)
+	encoder := yaml.NewEncoder(buf)
+	encoder.SetIndent(2)
+	err := encoder.Encode(o)
  • vendor/golang.org/x/tools/internal/event/keys/standard.go - 2
    • Incorrect comment for End variable · Line 14-14
      Incorrect comment for `End` variable. The comment says "Metric is a key used to indicate an event records metrics" but should describe the End variable instead. Fix: Change to "// End is a key used to indicate an event ends a span."
      code suggestion 
       @@ -14,1 +14,1 @@
-	// Metric is a key used to indicate an event records metrics.
+	// End is a key used to indicate an event ends a span.
    • Incorrect comment for Detach variable · Line 16-16
      Incorrect comment for `Detach` variable. The comment says "Metric is a key used to indicate an event records metrics" but should describe the Detach variable instead. Fix: Change to "// Detach is a key used to indicate an event detaches a span."
      code suggestion 
       @@ -16,1 +16,1 @@
-	// Metric is a key used to indicate an event records metrics.
+	// Detach is a key used to indicate an event detaches a span.
  • vendor/github.com/getkin/kin-openapi/openapi3/internalize_refs.go - 1
    • Incorrect buffer size allocation logic · Line 121-121
      The `cutDirectories` function incorrectly uses `len(ParameterInHeader)` to grow the string builder buffer. `ParameterInHeader` is a constant string "header" (6 characters), but the function should allocate buffer space based on the path length being processed. This causes inefficient memory allocation and potential performance issues when processing longer paths. Change to `sb.Grow(len(p))` to properly size the buffer.
      code suggestion 
       @@ -121,1 +121,1 @@
-	sb.Grow(len(ParameterInHeader))
+	sb.Grow(len(p))
  • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/apply.go - 1
    • Unreachable dead code error check · Line 99-99
      Dead code: Error check on lines 99-101 is unreachable because `p.Query(root)` doesn't return an error, but the code checks `err` which was already handled from the previous `o.NewPath()` call. This creates unreachable code that will never execute.
      code suggestion 
       @@ -99,3 +99,0 @@
-	if err != nil {
-		return err
-	}
Review Details
  • Files reviewed - 265 · Commit Range: d936815..4fe2e84
    • Wire.go
    • api/restHandler/BulkUpdateRestHandler.go
    • api/restHandler/BulkUpdateRestHandler_ent.go
    • api/router/BulkUpdateRouter.go
    • api/router/BulkUpdateRouter_ent.go
    • go.mod
    • go.sum
    • hack/tools.go
    • internal/sql/repository/chartConfig/ConfigMapRepository.go
    • pkg/bulkAction/bean/bean.go
    • pkg/bulkAction/repository/BulkEditRepository_ent.go
    • pkg/bulkAction/service/BulkUpdateService.go
    • pkg/bulkAction/service/BulkUpdateService_ent.go
    • pkg/bulkAction/utils/utils.go
    • pkg/team/repository/TeamRepository.go
    • pkg/team/repository/TeamRepository_ent.go
    • scripts/sql/34204000_bulk_edit_config.down.sql
    • scripts/sql/34204000_bulk_edit_config.up.sql
    • specs/bulkEdit/v1beta1/example.yml
    • specs/bulkEdit/v1beta1/schema.json
    • tests/pipeline/BulkUpdateService_test.go
    • tests/pipeline/ChartService_test.go
    • util/HttpUtil.go
    • util/context-utils.go
    • util/contextUtils.go
    • vendor/github.com/dprotaso/go-yit/.gitignore
    • vendor/github.com/dprotaso/go-yit/LICENSE
    • vendor/github.com/dprotaso/go-yit/README.md
    • vendor/github.com/dprotaso/go-yit/aggregates.go
    • vendor/github.com/dprotaso/go-yit/iterator.go
    • vendor/github.com/dprotaso/go-yit/predicates.go
    • vendor/github.com/getkin/kin-openapi/LICENSE
    • vendor/github.com/getkin/kin-openapi/openapi3/callback.go
    • vendor/github.com/getkin/kin-openapi/openapi3/components.go
    • vendor/github.com/getkin/kin-openapi/openapi3/contact.go
    • vendor/github.com/getkin/kin-openapi/openapi3/content.go
    • vendor/github.com/getkin/kin-openapi/openapi3/discriminator.go
    • vendor/github.com/getkin/kin-openapi/openapi3/doc.go
    • vendor/github.com/getkin/kin-openapi/openapi3/encoding.go
    • vendor/github.com/getkin/kin-openapi/openapi3/errors.go
    • vendor/github.com/getkin/kin-openapi/openapi3/example.go
    • vendor/github.com/getkin/kin-openapi/openapi3/example_validation.go
    • vendor/github.com/getkin/kin-openapi/openapi3/extension.go
    • vendor/github.com/getkin/kin-openapi/openapi3/external_docs.go
    • vendor/github.com/getkin/kin-openapi/openapi3/header.go
    • vendor/github.com/getkin/kin-openapi/openapi3/helpers.go
    • vendor/github.com/getkin/kin-openapi/openapi3/info.go
    • vendor/github.com/getkin/kin-openapi/openapi3/internalize_refs.go
    • vendor/github.com/getkin/kin-openapi/openapi3/license.go
    • vendor/github.com/getkin/kin-openapi/openapi3/link.go
    • vendor/github.com/getkin/kin-openapi/openapi3/loader.go
    • vendor/github.com/getkin/kin-openapi/openapi3/loader_uri_reader.go
    • vendor/github.com/getkin/kin-openapi/openapi3/maplike.go
    • vendor/github.com/getkin/kin-openapi/openapi3/marsh.go
    • vendor/github.com/getkin/kin-openapi/openapi3/media_type.go
    • vendor/github.com/getkin/kin-openapi/openapi3/openapi3.go
    • vendor/github.com/getkin/kin-openapi/openapi3/operation.go
    • vendor/github.com/getkin/kin-openapi/openapi3/parameter.go
    • vendor/github.com/getkin/kin-openapi/openapi3/path_item.go
    • vendor/github.com/getkin/kin-openapi/openapi3/paths.go
    • vendor/github.com/getkin/kin-openapi/openapi3/ref.go
    • vendor/github.com/getkin/kin-openapi/openapi3/refs.go
    • vendor/github.com/getkin/kin-openapi/openapi3/refs.tmpl
    • vendor/github.com/getkin/kin-openapi/openapi3/refs_test.tmpl
    • vendor/github.com/getkin/kin-openapi/openapi3/request_body.go
    • vendor/github.com/getkin/kin-openapi/openapi3/response.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_formats.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_pattern.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_validation_settings.go
    • vendor/github.com/getkin/kin-openapi/openapi3/security_requirements.go
    • vendor/github.com/getkin/kin-openapi/openapi3/security_scheme.go
    • vendor/github.com/getkin/kin-openapi/openapi3/serialization_method.go
    • vendor/github.com/getkin/kin-openapi/openapi3/server.go
    • vendor/github.com/getkin/kin-openapi/openapi3/tag.go
    • vendor/github.com/getkin/kin-openapi/openapi3/validation_options.go
    • vendor/github.com/getkin/kin-openapi/openapi3/visited.go
    • vendor/github.com/getkin/kin-openapi/openapi3/xml.go
    • vendor/github.com/invopop/yaml/.gitignore
    • vendor/github.com/invopop/yaml/.golangci.toml
    • vendor/github.com/invopop/yaml/LICENSE
    • vendor/github.com/invopop/yaml/README.md
    • vendor/github.com/invopop/yaml/fields.go
    • vendor/github.com/invopop/yaml/yaml.go
    • vendor/github.com/mohae/deepcopy/.gitignore
    • vendor/github.com/mohae/deepcopy/.travis.yml
    • vendor/github.com/mohae/deepcopy/LICENSE
    • vendor/github.com/mohae/deepcopy/README.md
    • vendor/github.com/mohae/deepcopy/deepcopy.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/LICENSE
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/cmd/oapi-codegen/oapi-codegen.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/codegen.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/configuration.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/extension.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/externalref.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/filter.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/inline.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas_v1.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/operations.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/prune.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/schema.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/template_helpers.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/additional-properties.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/client-with-responses.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/client.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/constants.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-register.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-wrappers.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-register.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-wrappers.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-register.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/imports.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/inline.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/param-types.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/request-bodies.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-echo.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-fiber-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-fiber.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-gin.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-http.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-iris-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-iris.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-responses.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/typedef.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/union-and-additional-properties.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/union.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/test_schema.json
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/test_spec.yaml
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/utils.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/inputmapping.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/isjson.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/loader.go
    • vendor/github.com/perimeterx/marshmallow/.gitignore
    • vendor/github.com/perimeterx/marshmallow/CHANGELOG.md
    • vendor/github.com/perimeterx/marshmallow/CODE_OF_CONDUCT.md
    • vendor/github.com/perimeterx/marshmallow/CONTRIBUTING.md
    • vendor/github.com/perimeterx/marshmallow/LICENSE
    • vendor/github.com/perimeterx/marshmallow/README.md
    • vendor/github.com/perimeterx/marshmallow/cache.go
    • vendor/github.com/perimeterx/marshmallow/doc.go
    • vendor/github.com/perimeterx/marshmallow/errors.go
    • vendor/github.com/perimeterx/marshmallow/options.go
    • vendor/github.com/perimeterx/marshmallow/reflection.go
    • vendor/github.com/perimeterx/marshmallow/unmarshal.go
    • vendor/github.com/perimeterx/marshmallow/unmarshal_from_json_map.go
    • vendor/github.com/speakeasy-api/openapi-overlay/LICENSE
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/loader/overlay.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/loader/spec.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/apply.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/compare.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parents.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parse.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/schema.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/utils.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/validate.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/LICENSE
    • vendor/github.com/vmware-labs/yaml-jsonpath/NOTICE
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/comparison.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/doc.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/filter.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/filter_parser.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/lexer.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/path.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/slicer.go
    • vendor/golang.org/x/mod/internal/lazyregexp/lazyre.go
    • vendor/golang.org/x/mod/module/module.go
    • vendor/golang.org/x/mod/module/pseudo.go
    • vendor/golang.org/x/text/cases/cases.go
    • vendor/golang.org/x/text/cases/context.go
    • vendor/golang.org/x/text/cases/fold.go
    • vendor/golang.org/x/text/cases/icu.go
    • vendor/golang.org/x/text/cases/info.go
    • vendor/golang.org/x/text/cases/map.go
    • vendor/golang.org/x/text/cases/tables10.0.0.go
    • vendor/golang.org/x/text/cases/tables11.0.0.go
    • vendor/golang.org/x/text/cases/tables12.0.0.go
    • vendor/golang.org/x/text/cases/tables13.0.0.go
    • vendor/golang.org/x/text/cases/tables15.0.0.go
    • vendor/golang.org/x/text/cases/tables9.0.0.go
    • vendor/golang.org/x/text/cases/trieval.go
    • vendor/golang.org/x/text/internal/internal.go
    • vendor/golang.org/x/text/internal/language/common.go
    • vendor/golang.org/x/text/internal/language/compact.go
    • vendor/golang.org/x/text/internal/language/compact/compact.go
    • vendor/golang.org/x/text/internal/language/compact/language.go
    • vendor/golang.org/x/text/internal/language/compact/parents.go
    • vendor/golang.org/x/text/internal/language/compact/tables.go
    • vendor/golang.org/x/text/internal/language/compact/tags.go
    • vendor/golang.org/x/text/internal/language/compose.go
    • vendor/golang.org/x/text/internal/language/coverage.go
    • vendor/golang.org/x/text/internal/language/language.go
    • vendor/golang.org/x/text/internal/language/lookup.go
    • vendor/golang.org/x/text/internal/language/match.go
    • vendor/golang.org/x/text/internal/language/parse.go
    • vendor/golang.org/x/text/internal/language/tables.go
    • vendor/golang.org/x/text/internal/language/tags.go
    • vendor/golang.org/x/text/internal/match.go
    • vendor/golang.org/x/text/internal/tag/tag.go
    • vendor/golang.org/x/text/language/coverage.go
    • vendor/golang.org/x/text/language/doc.go
    • vendor/golang.org/x/text/language/language.go
    • vendor/golang.org/x/text/language/match.go
    • vendor/golang.org/x/text/language/parse.go
    • vendor/golang.org/x/text/language/tables.go
    • vendor/golang.org/x/text/language/tags.go
    • vendor/golang.org/x/tools/LICENSE
    • vendor/golang.org/x/tools/PATENTS
    • vendor/golang.org/x/tools/go/ast/astutil/enclosing.go
    • vendor/golang.org/x/tools/go/ast/astutil/imports.go
    • vendor/golang.org/x/tools/go/ast/astutil/rewrite.go
    • vendor/golang.org/x/tools/go/ast/astutil/util.go
    • vendor/golang.org/x/tools/imports/forward.go
    • vendor/golang.org/x/tools/internal/event/core/event.go
    • vendor/golang.org/x/tools/internal/event/core/export.go
    • vendor/golang.org/x/tools/internal/event/core/fast.go
    • vendor/golang.org/x/tools/internal/event/doc.go
    • vendor/golang.org/x/tools/internal/event/event.go
    • vendor/golang.org/x/tools/internal/event/keys/keys.go
    • vendor/golang.org/x/tools/internal/event/keys/standard.go
    • vendor/golang.org/x/tools/internal/event/keys/util.go
    • vendor/golang.org/x/tools/internal/event/label/label.go
    • vendor/golang.org/x/tools/internal/gocommand/invoke.go
    • vendor/golang.org/x/tools/internal/gocommand/invoke_notunix.go
    • vendor/golang.org/x/tools/internal/gocommand/invoke_unix.go
    • vendor/golang.org/x/tools/internal/gocommand/vendor.go
    • vendor/golang.org/x/tools/internal/gocommand/version.go
    • vendor/golang.org/x/tools/internal/gopathwalk/walk.go
    • vendor/golang.org/x/tools/internal/imports/fix.go
    • vendor/golang.org/x/tools/internal/imports/imports.go
    • vendor/golang.org/x/tools/internal/imports/mod.go
    • vendor/golang.org/x/tools/internal/imports/mod_cache.go
    • vendor/golang.org/x/tools/internal/imports/sortimports.go
    • vendor/golang.org/x/tools/internal/imports/source.go
    • vendor/golang.org/x/tools/internal/imports/source_env.go
    • vendor/golang.org/x/tools/internal/imports/source_modindex.go
    • vendor/golang.org/x/tools/internal/modindex/directories.go
    • vendor/golang.org/x/tools/internal/modindex/index.go
    • vendor/golang.org/x/tools/internal/modindex/lookup.go
    • vendor/golang.org/x/tools/internal/modindex/modindex.go
    • vendor/golang.org/x/tools/internal/modindex/symbols.go
    • vendor/golang.org/x/tools/internal/modindex/types.go
    • vendor/golang.org/x/tools/internal/stdlib/deps.go
    • vendor/golang.org/x/tools/internal/stdlib/import.go
    • vendor/golang.org/x/tools/internal/stdlib/manifest.go
    • vendor/golang.org/x/tools/internal/stdlib/stdlib.go
    • vendor/modules.txt
    • wire_gen.go
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
    • GOVULNCHECK (Security Vulnerability) - ✖︎ Failed
    • OWASP (Security Vulnerability) - ✔︎ Successful
    • SNYK (Security Vulnerability) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

bito-code-review[bot]
bito-code-review bot reviewed Jul 15, 2025
View reviewed changes
@Shivam-nagar23
Copy link
Member

/review

@bito-code-review Bito Code Review
Copy link

Changelist by Bito

This pull request implements the following key changes.

Key Change Files Impacted
New Feature - New Feature Additions

- BulkUpdateRestHandler.go - Updated REST endpoints to support new Bulk Edit operations and deprecated old methods.

- BulkUpdateRestHandler_ent.go - Introduced a new interface for BulkEdit in the Ent variant with backward compatibility redirection.

- BulkUpdateRouter.go - Reorganized routing to segregate v1beta1 and v1beta2 endpoints for the new Bulk Edit API.

- BulkUpdateRouter_ent.go - Added stub routes for Ent-specific Bulk Edit endpoints to support the new feature.

- TeamRepository_ent.go - Introduced a new enterprise repository interface for team data management paving the way for future RBAC enhancements.

- schema.json - Added bulk edit schema definition with validation rules and property definitions.

- tag.go - Introduced the Tag object implementation with robust JSON and YAML marshalling.

- xml.go - Implemented XML support with custom marshalling and validation for OpenAPI 3 compliance.

- union.tmpl - Added new union handling functions with discriminator support for type resolution.

- test_schema.json - Introduced a new JSON schema defining a 'node' object for schema validation.

- test_spec.yaml - Added an OpenAPI specification file with expanded endpoints for testing and documentation.

- inputmapping.go - Added utility functions for parsing command-line mappings and handling escaped separators.

- isjson.go - Introduced a media type JSON validation function for checking JSON content types.

- loader.go - Implemented swagger loader utilities including support for external references and overlay application.
Feature Improvement - Bulk Edit and Go Command Improvements

- Wire.go - Refactored imports and dependency bindings by replacing legacy bulkUpdate repository with the new bulk edit repository.

- go.mod - Updated module dependencies with additions such as oapi-codegen, go-yit, and kin-openapi for enhanced bulk edit functionality.

- go.sum - Updated dependency versions and introduced new packages to support advanced bulk editing and validation.

- tools.go - Introduced a tooling file to manage build dependencies for generating API code.

- BulkUpdateService.go - Enhanced bulk update service implementation with added BulkEdit functionality and improved error logging.

- BulkUpdateService_ent.go - Introduced a new enterprise service interface and implementation for environment configuration validations.

- utils.go - Added the ApplyJsonPatch function for streamlined JSON patch operations.

- contextUtils.go - Added new context utility methods for enhanced token and email extraction during request handling.

- validation_options.go - Enhanced validation options enabling flexible configuration for OpenAPI document validation.

- unmarshal_from_json_map.go - Implemented advanced JSON map-based unmarshalling with improved performance and error handling.

- invoke.go - Enhanced go command runner with improved concurrency handling and debugging for hanging commands.

- invoke_notunix.go - Introduced non-Unix signal handling to force terminate hanging subprocesses.

- invoke_unix.go - Updated Unix-specific implementation to send SIGQUIT for capturing stack traces on hanging processes.

- version.go - Enhanced go version integration with robust version parsing and improved error management.

- wire_gen.go - Refactored bulk update dependency injection by updating the go:generate command, removing deprecated repository imports, and replacing bulkUpdate with the new bulk edit repository implementation.
Bug Fix - Team Repository Corrections

- TeamRepository.go - Fixed method signatures for FindByIds and GetConnection for proper handling of empty ID lists and correct database connection retrieval.
Documentation - Documentation and Spec Examples

- example.yml - Provided a detailed example YAML including sample configuration for bulk editing applications using deployment templates, config maps, and secrets.

- README.md - Updated documentation and usage examples for the YAML package.

- CHANGELOG.md - Updated changelog with recent notable changes.

- CODE_OF_CONDUCT.md - Added comprehensive Code of Conduct documentation.

- CONTRIBUTING.md - Provided guidelines on how to contribute to the project.

- LICENSE - Included the MIT License details for the project.

- README.md - Enhanced README with installation instructions, usage examples, and badges.

- LICENSE - Added license text for the Go tools package under a BSD-style license.

- PATENTS - Introduced additional IP rights information with patent license grant details.

- doc.go - Added package documentation for event telemetry packages.
Testing - Bulk Edit Service Test Updates

- BulkUpdateService_test.go - Updated test calls to use BulkEdit with context and updated user metadata retrieval for proper functionality.

- ChartService_test.go - Refactored test cases to use context-based BulkEdit calls and improved repository instantiation for enhanced test reliability.
Other Improvements - Other Improvements and Vendor Updates

- modules.txt - Updated vendor modules file with new dependency entries for text and tools packages, enhancing dependency tracking and supporting advanced bulk editing features.

- manifest.go - Expanded vendor manifest with extensive new definitions covering testing, template processing, scanning, and time functions to support enhanced system integration and vendor support.

- tables.go - Expanded Unicode tables with additional indexing entries for comprehensive language mapping and improved case conversion.

- tags.go - Added language tag definitions for various languages for enhanced locale processing.

- compose.go - Reorganized and improved language tag composition with enhanced token handling and extension processing.

- coverage.go - Implemented functions for retrieving base languages to streamline enumeration of supported languages.

- tables9.0.0.go - Revised Unicode 9.0.0 table with extensive new mapping block additions and refined case conversion.

- tables10.0.0.go - Expanded Unicode case mapping tables with extensive additions for comprehensive Unicode support.

- tables11.0.0.go - Expanded and refined the Unicode table with numerous new mapping blocks to fully support Unicode 11.0.0.

- tables12.0.0.go - Updated Unicode mapping table with new entries and extended blocks for enhanced case conversion precision.

- tables13.0.0.go - Extended and refined the Unicode mapping table with additional lookup blocks and optimized segmentation for Unicode 13.0.0 support.

- tables15.0.0.go - Updated Unicode mapping table with extensive new block entries and improved segmentation for better Unicode 15.0.0 case handling.

- trieval.go - Added generated code for case retrieval including trie definitions and lookup functions for enhanced Unicode case mapping.

- internal.go - Introduced internal utilities and non-exported functions to support text repository functionality.

- common.go - Added generated common language alias definitions and constants for improved language tag handling.

- compact.go - Defined a compact representation for language tags with unique core index calculations.

- compact.go - Implemented functions for managing compact language tags with parent lookup and unique indexing.

- language.go - Enhanced language tag processing with utilities for handling regional variants and improved tag conversions.

- parents.go - Added a generated mapping table for language parent relationships based on updated Unicode data.

- language.go - Enhanced language tag canonicalization functions with new additions and performance improvements.

- lookup.go - Improved lookup functionality with comprehensive language mapping, normalization, and region parsing.

- match.go - Updated matching logic for streamlined tag manipulation and improved likely subtag addition.

- tag.go - Optimized tag indexing with improved element retrieval and case comparison functions.

- coverage.go - Expanded support for retrieving tags, base languages, scripts, and regions with enhanced error handling.

- doc.go - Enhanced documentation clarifying language matching, canonicalization, and package usage.

- match.go - Refined matching implementation with detailed enhancements for better locale processing.

- parse.go - Improved language parsing and composition with robust error recovery and extended scanner utilities.

- tables.go - Updated language tables with new Unicode mapping entries and refined structure for enhanced locale processing.

- tags.go - Added new parsing functions and language tag constants for robust processing.

- validation_options.go - Enhanced validation options enabling flexible configuration for OpenAPI document validation.

- enclosing.go - Enhanced AST utility for efficient determination of node boundaries and precise child node sorting.

- imports.go - Added new import rewriting functions to manage AST imports with improved error handling.

- rewrite.go - Implemented comprehensive AST rewrite functionality with enhanced node traversal and manipulation.

- util.go - Introduced a utility to remove redundant parentheses from AST expressions for cleaner code.

- forward.go - Added import processing enhancements including formatting adjustments and dependency updates.

- event.go - Introduced core event functionality for telemetry with new event structures and utility methods.

- export.go - Implemented exporter enhancements for synchronous event handling and proper context propagation.

- fast.go - Optimized logging functions for faster event processing with minimized allocations.

- event.go - Integrated event exporter and logging functions with improved error handling and performance.

- keys.go - Added key management functionality for event labels supporting various data types.

- label.go - Enhanced label processing with new handler functions for comprehensive event metadata formatting.

- vendor.go - Improved vendor management with enhanced module resolution and dependency handling.

- walk.go - Added a concurrent file walker with symlink support for improved directory scanning in Go projects.

- fix.go - Enhanced concurrency handling and candidate selection in the import fix logic using context cancellation.

- imports.go - Enhanced formatting and grouping logic in the imports processor to better separate import sections.

- mod.go - Improved module resolution and caching with added functionality for module scanning and canonicalization.

- mod_cache.go - Implemented comprehensive caching and module cache scanning with robust error handling.

- sortimports.go - Refactored import sorting and merging functions for more efficient import handling.

- source.go - Added clearer source definitions for improved import resolution API clarity.

- source_env.go - Implemented ProcessEnvSource enhancements for better import environment handling and concurrency.

- source_modindex.go - Integrated a modindex-based source for better module reference resolution and caching.

- directories.go - Enhanced directory scanning and semantic sorting for improved module import indexing.

- index.go - Added comprehensive index management and error handling for module caching.

- lookup.go - Improved lookup functionality for module indexing with enhanced candidate search.

- modindex.go - Introduced new index creation and update mechanisms supporting advanced module caching features.

- symbols.go - Added symbol extraction and indexing capabilities to enhance module dependency analysis.

- types.go - Defined new types for handling module paths and relative references.

- deps.go - Added numerous dependency entries to the standard library dependency table for improved resolution.

- import.go - Introduced an API for building and traversing the standard library import graph with updated header comments.

- stdlib.go - Added a new stdlib package file including copyright headers, initialization routines, and symbol table construction for standard library exports.

bito-code-review[bot]
bito-code-review bot suggested changes Jul 15, 2025
View reviewed changes
Copy link

@bito-code-review bito-code-review bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bito is crafting review details...

@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 15, 2025 •
edited
Loading

Changelist by Bito

This pull request implements the following key changes.

Key Change Files Impacted
Bug Fix - Critical Bug Fixes

- DeploymentTemplateValidationService.go - Replaced fmt.Println with structured logging to properly report JSON schema parsing errors.

- PropertiesConfig.go - Modified GetLatestEnvironmentProperties to delegate to chartService, eliminating redundant error handling.

- HttpUtil.go - Removed redundant context token retrieval functions to streamline authentication.
New Feature - New Feature Additions

- BulkUpdateRouter.go - Reorganized router configurations by splitting routes into v1beta1, v1beta2, and enterprise categories to support the new bulk edit features.

- BulkUpdateRouter_ent.go - Added new routing stubs for enterprise-specific bulk edit endpoints with minimal implementations.

- go.mod - Incorporated new module dependencies and updated version constraints to support enhanced bulk edit capabilities.

- go.sum - Updated dependency checksums and introduced new module dependencies for bulk edit functionality.

- BulkEditRepository_ent.go - Introduced a new enterprise bulk edit repository interface with an Apache licensing header.

- ChartService.go - Added GetLatestEnvironmentProperties to retrieve the most recent environment properties, enhancing configuration management for deployment templates.

- chartEnvConfigOverride.go - Updated the chart environment override logic by adding a new method and improved JSON encoding support for handling chart configurations.

- 34204000_bulk_edit_config.down.sql - Added a down script to safely drop the bulk_edit_config table and its associated sequence when necessary.

- 34204000_bulk_edit_config.up.sql - Created and extended bulk_edit_config table and sequence with schema updates and additional configuration definitions for both v1beta1 and v1beta2.

- TeamRepository_ent.go - Added an enterprise-specific TeamRepository interface with proper licensing details.
Feature Improvement - Bulk Edit API Enhancements

- BulkUpdateRestHandler.go - Refactored bulk edit endpoints with updated naming conventions, authentication logic, and error handling to improve API reliability.

- BulkUpdateRestHandler_ent.go - Introduced new BulkEdit v1beta2 endpoints with backward compatibility measures for enhanced bulk edit operations.

- ConfigMapRepository.go - Added a ConfigModel interface and implemented methods for retrieving configuration data.

- bean.go - Refactored bulk edit payload types and removed obsolete error definitions.

- BulkUpdateService.go - Replaced legacy bulkUpdateRepository calls with bulkEditRepository and added new functions alongside improved error logging and authorization handling.

- BulkUpdateService_ent.go - Implemented an enterprise-specific bulk update service with extended error handling and modularity.

- TeamRepository.go - Refactored repository methods to improve team data retrieval logic and error handling.
Other Improvements - Dependency Injection Refactoring

- Wire.go - Updated repository bindings and import paths by replacing the legacy bulkUpdate with repository11, streamlining dependency management.

- tools.go - Added build tags and necessary imports for oapi-codegen to enhance development tooling.

- contextUtils.go - Replaced deprecated context utility functions by introducing a new contextUtils.go file that provides enhanced token and user metadata handling.

- patch.go - Implemented JSON patch application logic to improve JSON data modifications.

- wire_gen.go - Updated wiring for bulk edit repository and service implementations, replacing legacy bulkUpdate references with bulkEdit, and refining dependency injection.
Testing - Test Enhancements for Bulk Edit

- BulkUpdateService_test.go - Updated BulkUpdate tests to use BulkEdit with context and user metadata for improved test coverage.

- ChartService_test.go - Revised test cases to integrate context-based authentication in BulkUpdateDeploymentTemplate and update deployment template validations.

bito-code-review[bot]
bito-code-review bot suggested changes Jul 15, 2025
View reviewed changes
Copy link

@bito-code-review bito-code-review bot left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review Agent Run #2cc5cc

Actionable Suggestions - 30
  • specs/bulkEdit/v1beta1/schema.json - 1
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-middleware.tmpl - 1
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-register.tmpl - 1
    • Variable shadowing in anonymous function · Line 35-35
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/operations.go - 1
  • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/comparison.go - 1
    • Missing string ordering comparison logic · Line 59-63
  • vendor/golang.org/x/text/internal/language/tables.go - 7
  • vendor/github.com/getkin/kin-openapi/openapi3/marsh.go - 1
    • Incorrect string manipulation in error processing · Line 14-14
  • vendor/github.com/perimeterx/marshmallow/reflection.go - 1
    • Slice sharing causes incorrect field paths · Line 44-44
  • vendor/golang.org/x/text/language/coverage.go - 1
  • vendor/github.com/getkin/kin-openapi/openapi3/validation_options.go - 1
    • Missing validation option functions for example fields · Line 10-10
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas.go - 1
    • Incorrect AllOf field handling after merge · Line 121-121
  • vendor/github.com/perimeterx/marshmallow/unmarshal.go - 1
    • Slice append doesn't modify original parameter · Line 363-367
  • vendor/golang.org/x/tools/go/ast/astutil/enclosing.go - 2
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-handler.tmpl - 1
    • Conditional middleware registration breaks functionality · Line 13-23
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-responses.tmpl - 1
    • Unreachable template conditional logic · Line 14-14
  • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/utils.go - 1
    • JSONPath string escaping vulnerability · Line 9-9
  • vendor/github.com/speakeasy-api/openapi-overlay/pkg/loader/spec.go - 1
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-middleware.tmpl - 2
    • Incorrect path parameter extraction method · Line 21-21
    • Incorrect path parameter extraction for JSON · Line 24-24
  • vendor/golang.org/x/tools/internal/imports/sortimports.go - 1
    • Loop index not adjusted after slice deletion · Line 32-35
  • vendor/github.com/getkin/kin-openapi/openapi3/helpers.go - 1
  • vendor/golang.org/x/tools/internal/stdlib/manifest.go - 1
  • vendor/github.com/getkin/kin-openapi/openapi3/internalize_refs.go - 1
    • Incorrect buffer sizing with unrelated constant · Line 121-121
Additional Suggestions - 6
  • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parse.go - 1
    • Incorrect error message context · Line 43-43
      Error message incorrectly states 'failed to open overlay file' but the actual error is from `filepath.Abs()`. Change to 'failed to get absolute path for %q: %w'.
      code suggestion 
       @@ -43,1 +43,1 @@
-		return fmt.Errorf("failed to open overlay file at path %q: %w", path, err)
+		return fmt.Errorf("failed to get absolute path for %q: %w", path, err)
  • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/loader.go - 1
    • Misleading error message for LoadFromDataWithPath · Line 96-96
      The error message incorrectly states 'failed to serialize' when the actual operation that failed is loading/deserializing the overlaid specification using `LoadFromDataWithPath`. This misleading error message will confuse developers during debugging. Change to 'failed to load Overlay'd specification'.
      code suggestion 
       @@ -96,1 +96,1 @@
-		return nil, fmt.Errorf("failed to serialize Overlay'd specification %#v: %v", opts.Path, err)
+		return nil, fmt.Errorf("failed to load Overlay'd specification %#v: %v", opts.Path, err)
  • vendor/golang.org/x/tools/internal/event/keys/keys.go - 3
    • Incorrect NewFloat64 documentation comment · Line 432-432
      The documentation comment for `NewFloat64` incorrectly states it creates a key for int64 values, but it actually creates a key for float64 values. This misleading documentation could confuse developers using this API.
      code suggestion 
       @@ -432,1 +432,1 @@
-// NewFloat64 creates a new Key for int64 values.
+// NewFloat64 creates a new Key for float64 values.
    • Incorrect NewString documentation comment · Line 468-468
      The documentation comment for `NewString` incorrectly states it creates a key for int64 values, but it actually creates a key for string values. This misleading documentation could confuse developers using this API.
      code suggestion 
       @@ -468,1 +468,1 @@
-// NewString creates a new Key for int64 values.
+// NewString creates a new Key for string values.
    • Incorrect NewError documentation comment · Line 537-537
      The documentation comment for `NewError` incorrectly states it creates a key for int64 values, but it actually creates a key for error values. This misleading documentation could confuse developers using this API.
      code suggestion 
       @@ -537,1 +537,1 @@
-// NewError creates a new Key for int64 values.
+// NewError creates a new Key for error values.
  • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/apply.go - 1
    • Dead code after Query call · Line 99-99
      Dead code detected: error check after `p.Query(root)` call will never be reached because `p.Query()` only returns a slice of nodes, not an error. This unreachable code should be removed to prevent confusion and maintain code clarity.
      code suggestion 
       @@ -99,3 +99,0 @@
-	if err != nil {
-		return err
-	}
Filtered by Review Rules

Bito filtered these suggestions based on rules created automatically for your feedback. Manage rules.

  • scripts/sql/34204000_bulk_edit_config.down.sql - 1
    • Migration sequence name mismatch · Line 7-7
  • vendor/golang.org/x/text/internal/language/compact.go - 1
    • RegionID bit mask truncation issue · Line 26-26
  • vendor/github.com/invopop/yaml/.golangci.toml - 1
    • Duplicate linter configuration entry · Line 9-9
  • vendor/golang.org/x/tools/internal/stdlib/manifest.go - 1
Review Details
  • Files reviewed - 265 · Commit Range: d936815..4fe2e84
    • Wire.go
    • api/restHandler/BulkUpdateRestHandler.go
    • api/restHandler/BulkUpdateRestHandler_ent.go
    • api/router/BulkUpdateRouter.go
    • api/router/BulkUpdateRouter_ent.go
    • go.mod
    • go.sum
    • hack/tools.go
    • internal/sql/repository/chartConfig/ConfigMapRepository.go
    • pkg/bulkAction/bean/bean.go
    • pkg/bulkAction/repository/BulkEditRepository_ent.go
    • pkg/bulkAction/service/BulkUpdateService.go
    • pkg/bulkAction/service/BulkUpdateService_ent.go
    • pkg/bulkAction/utils/utils.go
    • pkg/team/repository/TeamRepository.go
    • pkg/team/repository/TeamRepository_ent.go
    • scripts/sql/34204000_bulk_edit_config.down.sql
    • scripts/sql/34204000_bulk_edit_config.up.sql
    • specs/bulkEdit/v1beta1/example.yml
    • specs/bulkEdit/v1beta1/schema.json
    • tests/pipeline/BulkUpdateService_test.go
    • tests/pipeline/ChartService_test.go
    • util/HttpUtil.go
    • util/context-utils.go
    • util/contextUtils.go
    • vendor/github.com/dprotaso/go-yit/.gitignore
    • vendor/github.com/dprotaso/go-yit/LICENSE
    • vendor/github.com/dprotaso/go-yit/README.md
    • vendor/github.com/dprotaso/go-yit/aggregates.go
    • vendor/github.com/dprotaso/go-yit/iterator.go
    • vendor/github.com/dprotaso/go-yit/predicates.go
    • vendor/github.com/getkin/kin-openapi/LICENSE
    • vendor/github.com/getkin/kin-openapi/openapi3/callback.go
    • vendor/github.com/getkin/kin-openapi/openapi3/components.go
    • vendor/github.com/getkin/kin-openapi/openapi3/contact.go
    • vendor/github.com/getkin/kin-openapi/openapi3/content.go
    • vendor/github.com/getkin/kin-openapi/openapi3/discriminator.go
    • vendor/github.com/getkin/kin-openapi/openapi3/doc.go
    • vendor/github.com/getkin/kin-openapi/openapi3/encoding.go
    • vendor/github.com/getkin/kin-openapi/openapi3/errors.go
    • vendor/github.com/getkin/kin-openapi/openapi3/example.go
    • vendor/github.com/getkin/kin-openapi/openapi3/example_validation.go
    • vendor/github.com/getkin/kin-openapi/openapi3/extension.go
    • vendor/github.com/getkin/kin-openapi/openapi3/external_docs.go
    • vendor/github.com/getkin/kin-openapi/openapi3/header.go
    • vendor/github.com/getkin/kin-openapi/openapi3/helpers.go
    • vendor/github.com/getkin/kin-openapi/openapi3/info.go
    • vendor/github.com/getkin/kin-openapi/openapi3/internalize_refs.go
    • vendor/github.com/getkin/kin-openapi/openapi3/license.go
    • vendor/github.com/getkin/kin-openapi/openapi3/link.go
    • vendor/github.com/getkin/kin-openapi/openapi3/loader.go
    • vendor/github.com/getkin/kin-openapi/openapi3/loader_uri_reader.go
    • vendor/github.com/getkin/kin-openapi/openapi3/maplike.go
    • vendor/github.com/getkin/kin-openapi/openapi3/marsh.go
    • vendor/github.com/getkin/kin-openapi/openapi3/media_type.go
    • vendor/github.com/getkin/kin-openapi/openapi3/openapi3.go
    • vendor/github.com/getkin/kin-openapi/openapi3/operation.go
    • vendor/github.com/getkin/kin-openapi/openapi3/parameter.go
    • vendor/github.com/getkin/kin-openapi/openapi3/path_item.go
    • vendor/github.com/getkin/kin-openapi/openapi3/paths.go
    • vendor/github.com/getkin/kin-openapi/openapi3/ref.go
    • vendor/github.com/getkin/kin-openapi/openapi3/refs.go
    • vendor/github.com/getkin/kin-openapi/openapi3/refs.tmpl
    • vendor/github.com/getkin/kin-openapi/openapi3/refs_test.tmpl
    • vendor/github.com/getkin/kin-openapi/openapi3/request_body.go
    • vendor/github.com/getkin/kin-openapi/openapi3/response.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_formats.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_pattern.go
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_validation_settings.go
    • vendor/github.com/getkin/kin-openapi/openapi3/security_requirements.go
    • vendor/github.com/getkin/kin-openapi/openapi3/security_scheme.go
    • vendor/github.com/getkin/kin-openapi/openapi3/serialization_method.go
    • vendor/github.com/getkin/kin-openapi/openapi3/server.go
    • vendor/github.com/getkin/kin-openapi/openapi3/tag.go
    • vendor/github.com/getkin/kin-openapi/openapi3/validation_options.go
    • vendor/github.com/getkin/kin-openapi/openapi3/visited.go
    • vendor/github.com/getkin/kin-openapi/openapi3/xml.go
    • vendor/github.com/invopop/yaml/.gitignore
    • vendor/github.com/invopop/yaml/.golangci.toml
    • vendor/github.com/invopop/yaml/LICENSE
    • vendor/github.com/invopop/yaml/README.md
    • vendor/github.com/invopop/yaml/fields.go
    • vendor/github.com/invopop/yaml/yaml.go
    • vendor/github.com/mohae/deepcopy/.gitignore
    • vendor/github.com/mohae/deepcopy/.travis.yml
    • vendor/github.com/mohae/deepcopy/LICENSE
    • vendor/github.com/mohae/deepcopy/README.md
    • vendor/github.com/mohae/deepcopy/deepcopy.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/LICENSE
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/cmd/oapi-codegen/oapi-codegen.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/codegen.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/configuration.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/extension.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/externalref.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/filter.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/inline.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas_v1.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/operations.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/prune.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/schema.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/template_helpers.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/additional-properties.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/client-with-responses.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/client.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/constants.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-register.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-wrappers.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-register.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-wrappers.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-register.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/imports.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/inline.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/param-types.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/request-bodies.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-handler.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-middleware.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-echo.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-fiber-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-fiber.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-gin.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-http.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-iris-interface.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-iris.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-responses.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/typedef.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/union-and-additional-properties.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/union.tmpl
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/test_schema.json
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/test_spec.yaml
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/utils.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/inputmapping.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/isjson.go
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/loader.go
    • vendor/github.com/perimeterx/marshmallow/.gitignore
    • vendor/github.com/perimeterx/marshmallow/CHANGELOG.md
    • vendor/github.com/perimeterx/marshmallow/CODE_OF_CONDUCT.md
    • vendor/github.com/perimeterx/marshmallow/CONTRIBUTING.md
    • vendor/github.com/perimeterx/marshmallow/LICENSE
    • vendor/github.com/perimeterx/marshmallow/README.md
    • vendor/github.com/perimeterx/marshmallow/cache.go
    • vendor/github.com/perimeterx/marshmallow/doc.go
    • vendor/github.com/perimeterx/marshmallow/errors.go
    • vendor/github.com/perimeterx/marshmallow/options.go
    • vendor/github.com/perimeterx/marshmallow/reflection.go
    • vendor/github.com/perimeterx/marshmallow/unmarshal.go
    • vendor/github.com/perimeterx/marshmallow/unmarshal_from_json_map.go
    • vendor/github.com/speakeasy-api/openapi-overlay/LICENSE
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/loader/overlay.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/loader/spec.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/apply.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/compare.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parents.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parse.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/schema.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/utils.go
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/validate.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/LICENSE
    • vendor/github.com/vmware-labs/yaml-jsonpath/NOTICE
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/comparison.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/doc.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/filter.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/filter_parser.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/lexer.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/path.go
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/slicer.go
    • vendor/golang.org/x/mod/internal/lazyregexp/lazyre.go
    • vendor/golang.org/x/mod/module/module.go
    • vendor/golang.org/x/mod/module/pseudo.go
    • vendor/golang.org/x/text/cases/cases.go
    • vendor/golang.org/x/text/cases/context.go
    • vendor/golang.org/x/text/cases/fold.go
    • vendor/golang.org/x/text/cases/icu.go
    • vendor/golang.org/x/text/cases/info.go
    • vendor/golang.org/x/text/cases/map.go
    • vendor/golang.org/x/text/cases/tables10.0.0.go
    • vendor/golang.org/x/text/cases/tables11.0.0.go
    • vendor/golang.org/x/text/cases/tables12.0.0.go
    • vendor/golang.org/x/text/cases/tables13.0.0.go
    • vendor/golang.org/x/text/cases/tables15.0.0.go
    • vendor/golang.org/x/text/cases/tables9.0.0.go
    • vendor/golang.org/x/text/cases/trieval.go
    • vendor/golang.org/x/text/internal/internal.go
    • vendor/golang.org/x/text/internal/language/common.go
    • vendor/golang.org/x/text/internal/language/compact.go
    • vendor/golang.org/x/text/internal/language/compact/compact.go
    • vendor/golang.org/x/text/internal/language/compact/language.go
    • vendor/golang.org/x/text/internal/language/compact/parents.go
    • vendor/golang.org/x/text/internal/language/compact/tables.go
    • vendor/golang.org/x/text/internal/language/compact/tags.go
    • vendor/golang.org/x/text/internal/language/compose.go
    • vendor/golang.org/x/text/internal/language/coverage.go
    • vendor/golang.org/x/text/internal/language/language.go
    • vendor/golang.org/x/text/internal/language/lookup.go
    • vendor/golang.org/x/text/internal/language/match.go
    • vendor/golang.org/x/text/internal/language/parse.go
    • vendor/golang.org/x/text/internal/language/tables.go
    • vendor/golang.org/x/text/internal/language/tags.go
    • vendor/golang.org/x/text/internal/match.go
    • vendor/golang.org/x/text/internal/tag/tag.go
    • vendor/golang.org/x/text/language/coverage.go
    • vendor/golang.org/x/text/language/doc.go
    • vendor/golang.org/x/text/language/language.go
    • vendor/golang.org/x/text/language/match.go
    • vendor/golang.org/x/text/language/parse.go
    • vendor/golang.org/x/text/language/tables.go
    • vendor/golang.org/x/text/language/tags.go
    • vendor/golang.org/x/tools/LICENSE
    • vendor/golang.org/x/tools/PATENTS
    • vendor/golang.org/x/tools/go/ast/astutil/enclosing.go
    • vendor/golang.org/x/tools/go/ast/astutil/imports.go
    • vendor/golang.org/x/tools/go/ast/astutil/rewrite.go
    • vendor/golang.org/x/tools/go/ast/astutil/util.go
    • vendor/golang.org/x/tools/imports/forward.go
    • vendor/golang.org/x/tools/internal/event/core/event.go
    • vendor/golang.org/x/tools/internal/event/core/export.go
    • vendor/golang.org/x/tools/internal/event/core/fast.go
    • vendor/golang.org/x/tools/internal/event/doc.go
    • vendor/golang.org/x/tools/internal/event/event.go
    • vendor/golang.org/x/tools/internal/event/keys/keys.go
    • vendor/golang.org/x/tools/internal/event/keys/standard.go
    • vendor/golang.org/x/tools/internal/event/keys/util.go
    • vendor/golang.org/x/tools/internal/event/label/label.go
    • vendor/golang.org/x/tools/internal/gocommand/invoke.go
    • vendor/golang.org/x/tools/internal/gocommand/invoke_notunix.go
    • vendor/golang.org/x/tools/internal/gocommand/invoke_unix.go
    • vendor/golang.org/x/tools/internal/gocommand/vendor.go
    • vendor/golang.org/x/tools/internal/gocommand/version.go
    • vendor/golang.org/x/tools/internal/gopathwalk/walk.go
    • vendor/golang.org/x/tools/internal/imports/fix.go
    • vendor/golang.org/x/tools/internal/imports/imports.go
    • vendor/golang.org/x/tools/internal/imports/mod.go
    • vendor/golang.org/x/tools/internal/imports/mod_cache.go
    • vendor/golang.org/x/tools/internal/imports/sortimports.go
    • vendor/golang.org/x/tools/internal/imports/source.go
    • vendor/golang.org/x/tools/internal/imports/source_env.go
    • vendor/golang.org/x/tools/internal/imports/source_modindex.go
    • vendor/golang.org/x/tools/internal/modindex/directories.go
    • vendor/golang.org/x/tools/internal/modindex/index.go
    • vendor/golang.org/x/tools/internal/modindex/lookup.go
    • vendor/golang.org/x/tools/internal/modindex/modindex.go
    • vendor/golang.org/x/tools/internal/modindex/symbols.go
    • vendor/golang.org/x/tools/internal/modindex/types.go
    • vendor/golang.org/x/tools/internal/stdlib/deps.go
    • vendor/golang.org/x/tools/internal/stdlib/import.go
    • vendor/golang.org/x/tools/internal/stdlib/manifest.go
    • vendor/golang.org/x/tools/internal/stdlib/stdlib.go
    • vendor/modules.txt
    • wire_gen.go
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
    • GOVULNCHECK (Security Vulnerability) - ✖︎ Failed
    • OWASP (Security Vulnerability) - ✔︎ Successful
    • SNYK (Security Vulnerability) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

bito-code-review[bot]
bito-code-review bot reviewed Jul 15, 2025
View reviewed changes
bito-code-review[bot]
bito-code-review bot suggested changes Jul 15, 2025
View reviewed changes
Copy link

@bito-code-review bito-code-review bot left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review Agent Run #04a850

Actionable Suggestions - 2
  • scripts/sql/34204000_bulk_edit_config.up.sql - 1
  • pkg/bulkAction/service/BulkUpdateService_ent.go - 1
    • Constructor returns nil causing nil pointer · Line 40-40
Review Details
  • Files reviewed - 27 · Commit Range: d936815..04ec183
    • Wire.go
    • api/restHandler/BulkUpdateRestHandler.go
    • api/restHandler/BulkUpdateRestHandler_ent.go
    • api/router/BulkUpdateRouter.go
    • api/router/BulkUpdateRouter_ent.go
    • go.mod
    • go.sum
    • hack/tools.go
    • internal/sql/repository/chartConfig/ConfigMapRepository.go
    • pkg/bulkAction/bean/bean.go
    • pkg/bulkAction/repository/BulkEditRepository_ent.go
    • pkg/bulkAction/service/BulkUpdateService.go
    • pkg/bulkAction/service/BulkUpdateService_ent.go
    • pkg/bulkAction/utils/utils.go
    • pkg/chart/ChartService.go
    • pkg/deployment/manifest/deploymentTemplate/validator/DeploymentTemplateValidationService.go
    • pkg/pipeline/PropertiesConfig.go
    • pkg/team/repository/TeamRepository.go
    • pkg/team/repository/TeamRepository_ent.go
    • scripts/sql/34204000_bulk_edit_config.down.sql
    • scripts/sql/34204000_bulk_edit_config.up.sql
    • tests/pipeline/BulkUpdateService_test.go
    • tests/pipeline/ChartService_test.go
    • util/HttpUtil.go
    • util/context-utils.go
    • util/contextUtils.go
    • wire_gen.go
  • Files skipped - 241
    • specs/bulkEdit/v1beta1/example.yml - Reason: Filter setting
    • specs/bulkEdit/v1beta1/schema.json - Reason: Filter setting
    • vendor/github.com/dprotaso/go-yit/.gitignore - Reason: Filter setting
    • vendor/github.com/dprotaso/go-yit/LICENSE - Reason: Filter setting
    • vendor/github.com/dprotaso/go-yit/README.md - Reason: Filter setting
    • vendor/github.com/dprotaso/go-yit/aggregates.go - Reason: Filter setting
    • vendor/github.com/dprotaso/go-yit/iterator.go - Reason: Filter setting
    • vendor/github.com/dprotaso/go-yit/predicates.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/LICENSE - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/callback.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/components.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/contact.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/content.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/discriminator.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/doc.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/encoding.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/errors.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/example.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/example_validation.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/extension.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/external_docs.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/header.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/helpers.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/info.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/internalize_refs.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/license.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/link.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/loader.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/loader_uri_reader.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/maplike.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/marsh.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/media_type.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/openapi3.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/operation.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/parameter.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/path_item.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/paths.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/ref.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/refs.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/refs.tmpl - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/refs_test.tmpl - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/request_body.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/response.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/schema.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_formats.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_pattern.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/schema_validation_settings.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/security_requirements.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/security_scheme.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/serialization_method.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/server.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/tag.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/validation_options.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/visited.go - Reason: Filter setting
    • vendor/github.com/getkin/kin-openapi/openapi3/xml.go - Reason: Filter setting
    • vendor/github.com/invopop/yaml/.gitignore - Reason: Filter setting
    • vendor/github.com/invopop/yaml/.golangci.toml - Reason: Filter setting
    • vendor/github.com/invopop/yaml/LICENSE - Reason: Filter setting
    • vendor/github.com/invopop/yaml/README.md - Reason: Filter setting
    • vendor/github.com/invopop/yaml/fields.go - Reason: Filter setting
    • vendor/github.com/invopop/yaml/yaml.go - Reason: Filter setting
    • vendor/github.com/mohae/deepcopy/.gitignore - Reason: Filter setting
    • vendor/github.com/mohae/deepcopy/.travis.yml - Reason: Filter setting
    • vendor/github.com/mohae/deepcopy/LICENSE - Reason: Filter setting
    • vendor/github.com/mohae/deepcopy/README.md - Reason: Filter setting
    • vendor/github.com/mohae/deepcopy/deepcopy.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/LICENSE - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/cmd/oapi-codegen/oapi-codegen.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/codegen.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/configuration.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/extension.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/externalref.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/filter.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/inline.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/merge_schemas_v1.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/operations.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/prune.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/schema.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/template_helpers.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/additional-properties.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-handler.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/chi/chi-middleware.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/client-with-responses.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/client.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/constants.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-register.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/echo/echo-wrappers.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-handler.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/fiber/fiber-middleware.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-register.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gin/gin-wrappers.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-middleware.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/gorilla/gorilla-register.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/imports.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/inline.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-handler.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/iris/iris-middleware.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/param-types.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/request-bodies.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-handler.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/stdhttp/std-http-middleware.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-echo.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-fiber-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-fiber.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-gin.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-http.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-iris-interface.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-iris.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/strict/strict-responses.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/typedef.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/union-and-additional-properties.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/templates/union.tmpl - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/test_schema.json - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/test_spec.yaml - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/codegen/utils.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/inputmapping.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/isjson.go - Reason: Filter setting
    • vendor/github.com/oapi-codegen/oapi-codegen/v2/pkg/util/loader.go - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/.gitignore - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/CHANGELOG.md - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/CODE_OF_CONDUCT.md - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/CONTRIBUTING.md - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/LICENSE - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/README.md - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/cache.go - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/doc.go - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/errors.go - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/options.go - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/reflection.go - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/unmarshal.go - Reason: Filter setting
    • vendor/github.com/perimeterx/marshmallow/unmarshal_from_json_map.go - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/LICENSE - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/loader/overlay.go - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/loader/spec.go - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/apply.go - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/compare.go - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parents.go - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/parse.go - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/schema.go - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/utils.go - Reason: Filter setting
    • vendor/github.com/speakeasy-api/openapi-overlay/pkg/overlay/validate.go - Reason: Filter setting
    • vendor/github.com/vmware-labs/yaml-jsonpath/LICENSE - Reason: Filter setting
    • vendor/github.com/vmware-labs/yaml-jsonpath/NOTICE - Reason: Filter setting
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/comparison.go - Reason: Filter setting
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/doc.go - Reason: Filter setting
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/filter.go - Reason: Filter setting
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/filter_parser.go - Reason: Filter setting
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/lexer.go - Reason: Filter setting
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/path.go - Reason: Filter setting
    • vendor/github.com/vmware-labs/yaml-jsonpath/pkg/yamlpath/slicer.go - Reason: Filter setting
    • vendor/golang.org/x/mod/internal/lazyregexp/lazyre.go - Reason: Filter setting
    • vendor/golang.org/x/mod/module/module.go - Reason: Filter setting
    • vendor/golang.org/x/mod/module/pseudo.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/cases.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/context.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/fold.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/icu.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/info.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/map.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/tables10.0.0.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/tables11.0.0.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/tables12.0.0.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/tables13.0.0.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/tables15.0.0.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/tables9.0.0.go - Reason: Filter setting
    • vendor/golang.org/x/text/cases/trieval.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/internal.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/common.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/compact.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/compact/compact.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/compact/language.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/compact/parents.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/compact/tables.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/compact/tags.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/compose.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/coverage.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/language.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/lookup.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/match.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/parse.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/tables.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/language/tags.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/match.go - Reason: Filter setting
    • vendor/golang.org/x/text/internal/tag/tag.go - Reason: Filter setting
    • vendor/golang.org/x/text/language/coverage.go - Reason: Filter setting
    • vendor/golang.org/x/text/language/doc.go - Reason: Filter setting
    • vendor/golang.org/x/text/language/language.go - Reason: Filter setting
    • vendor/golang.org/x/text/language/match.go - Reason: Filter setting
    • vendor/golang.org/x/text/language/parse.go - Reason: Filter setting
    • vendor/golang.org/x/text/language/tables.go - Reason: Filter setting
    • vendor/golang.org/x/text/language/tags.go - Reason: Filter setting
    • vendor/golang.org/x/tools/LICENSE - Reason: Filter setting
    • vendor/golang.org/x/tools/PATENTS - Reason: Filter setting
    • vendor/golang.org/x/tools/go/ast/astutil/enclosing.go - Reason: Filter setting
    • vendor/golang.org/x/tools/go/ast/astutil/imports.go - Reason: Filter setting
    • vendor/golang.org/x/tools/go/ast/astutil/rewrite.go - Reason: Filter setting
    • vendor/golang.org/x/tools/go/ast/astutil/util.go - Reason: Filter setting
    • vendor/golang.org/x/tools/imports/forward.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/event/core/event.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/event/core/export.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/event/core/fast.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/event/doc.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/event/event.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/event/keys/keys.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/event/keys/standard.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/event/keys/util.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/event/label/label.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/gocommand/invoke.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/gocommand/invoke_notunix.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/gocommand/invoke_unix.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/gocommand/vendor.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/gocommand/version.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/gopathwalk/walk.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/imports/fix.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/imports/imports.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/imports/mod.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/imports/mod_cache.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/imports/sortimports.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/imports/source.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/imports/source_env.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/imports/source_modindex.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/modindex/directories.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/modindex/index.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/modindex/lookup.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/modindex/modindex.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/modindex/symbols.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/modindex/types.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/stdlib/deps.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/stdlib/import.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/stdlib/manifest.go - Reason: Filter setting
    • vendor/golang.org/x/tools/internal/stdlib/stdlib.go - Reason: Filter setting
    • vendor/modules.txt - Reason: Filter setting
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
    • SNYK (Security Vulnerability) - ✔︎ Successful
    • GOVULNCHECK (Security Vulnerability) - ✖︎ Failed
    • OWASP (Security Vulnerability) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 16, 2025 •
edited
Loading

Code Review Agent Run #04ecd1

Actionable Suggestions - 0
Filtered by Review Rules

Bito filtered these suggestions based on rules created automatically for your feedback. Manage rules.

  • scripts/sql/34204000_bulk_edit_config.up.sql - 1
    • Database schema validation mismatch · Line 95-95
Review Details
  • Files reviewed - 1 · Commit Range: 04ec183..19bb1a0
    • scripts/sql/34204000_bulk_edit_config.up.sql
  • Files skipped - 1
    • specs/bulkEdit/v1beta1/schema.json - Reason: Filter setting
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 20, 2025 •
edited
Loading

Code Review Agent Run #0c4fce

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 19bb1a0..45f135f
    • scripts/sql/34204000_bulk_edit_config.up.sql
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 20, 2025 •
edited
Loading

Bito Review Skipped - No Changes Detected

Bito didn't review this pull request because we did not detect any changes in the pull request to review.

@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 21, 2025 •
edited
Loading

Code Review Agent Run #7e9dc9

Actionable Suggestions - 0
Review Details
  • Files reviewed - 1 · Commit Range: 45f135f..1078d3f
    • scripts/sql/34204000_bulk_edit_config.up.sql
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

kartik-579
kartik-579 requested changes Jul 21, 2025
View reviewed changes
vendor/github.com/oapi-codegen/oapi-codegen/v2/LICENSE Outdated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add developer(in develop-guide dir) doc for usage of this

@bito-code-review Bito Code Review
Copy link

bito-code-review bot commented Jul 22, 2025 •
edited
Loading

Code Review Agent Run #e4fbed

Actionable Suggestions - 0
Review Details
  • Files reviewed - 2 · Commit Range: 1078d3f..3b3dba8
    • api/restHandler/BulkUpdateRestHandler.go
    • pkg/bulkAction/utils/utils.go
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
20 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.1% Duplication on New Code

See analysis details on SonarQube Cloud

bito-code-review[bot]
bito-code-review bot suggested changes Jul 22, 2025
View reviewed changes
Copy link

@bito-code-review bito-code-review bot left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review Agent Run #92fcaa

Actionable Suggestions - 1
  • scripts/sql/34204000_bulk_edit_config.up.sql - 1
Review Details
  • Files reviewed - 6 · Commit Range: 3b3dba8..8d97acc
    • scripts/sql/34204000_bulk_edit_config.up.sql
    • pkg/bulkAction/service/BulkUpdateService.go
    • pkg/bulkAction/utils/utils.go
    • util/json/patch.go
    • pkg/chart/ChartService.go
    • pkg/deployment/manifest/deploymentTemplate/read/chartEnvConfigOverride.go
  • Files skipped - 0
  • Tools
    • Whispers (Secret Scanner) - ✔︎ Successful
    • Detect-secrets (Secret Scanner) - ✔︎ Successful
Bito Usage Guide

Commands

Type the following command in the pull request comment and save the comment.

  • /review - Manually triggers a full AI review.

  • /pause - Pauses automatic reviews on this pull request.

  • /resume - Resumes automatic reviews.

  • /resolve - Marks all Bito-posted review comments as resolved.

  • /abort - Cancels all in-progress reviews.

Refer to the documentation for additional commands.

Configuration

This repository uses Default Agent You can customize the agent settings here or contact your Bito workspace admin at [email protected].

Documentation & Help

AI Code Review powered by Bito Logo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kartik-579 kartik-579 kartik-579 requested changes

@bito-code-review bito-code-review[bot] bito-code-review[bot] requested changes

@vikramdevtron vikramdevtron Awaiting requested review from vikramdevtron vikramdevtron is a code owner

@nishant-d nishant-d Awaiting requested review from nishant-d nishant-d is a code owner

@prakarsh-dt prakarsh-dt Awaiting requested review from prakarsh-dt prakarsh-dt is a code owner

@vivek-devtron vivek-devtron Awaiting requested review from vivek-devtron vivek-devtron is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

@Ash-exp Ash-exp

Labels
PR:Ready-to-Review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Ash-exp @Shivam-nagar23 @kartik-579