DeepDocs: Docs updates triggered by 7239e31 in main#4
Open
deepdocsai-dev[bot] wants to merge 1 commit intomainfrom
Open
DeepDocs: Docs updates triggered by 7239e31 in main#4deepdocsai-dev[bot] wants to merge 1 commit intomainfrom
deepdocsai-dev[bot] wants to merge 1 commit intomainfrom
Conversation
- docs/cli/commands/extraction-create.mdx: Documented CLI usage flag for 'regex' parameter is incorrect ('--attribute'); CLI help section shows '--regex'. | Documented CLI usage placeholder for 'condition' parameter is incorrect ('<condition-attribute>'); does not match CLI help section.
- docs/deployment/secret-store.mdx: Documentation incorrectly states 'four primary secret manager types' but the enum lists six: FILE, AWS, GCP, K8S, VAULT, DB | Factory supported types miss AWS and DB which have dedicated classes and are in the enum | Environment variable for tags is SECRET_MANAGER_TAGS in code, not AWS_SECRET_MANAGER_TAGS | Kubernetes namespace defaults to 'default' in code, not '.metadata.namespace'
- docs/deployment/provision/provider.mdx: The KEEP_PROVIDERS environment variable example uses inconsistent configuration values for the ClickHouse provider compared to the preceding JSON example (port 4321 vs 1234, password '1234' vs 'keep', database 'keepdb' vs 'keep-db')
- docs/providers/documentation/http-provider.mdx: Constructor requires `context_manager` as first parameter, but documentation only mentions `provider_id` and `ProviderConfig`
- docs/providers/documentation/opensearchserverless-provider.mdx: Documentation incorrectly lists the parameter for document ID as 'id', but the code signature uses 'doc_id'
- docs/cli/commands/cli-api.mdx: CLI help options list is incomplete; code signature for api() includes 'host: str' and 'port: int' parameters corresponding to --host and --port options
- docs/authentication/okta.md: Documentation lists 'User and group management through Okta' as a supported feature, but all provided IdentityManager methods (create_role, delete_role, get_roles, get_users, delete_user, update_user, get_groups) explicitly disable management by logging 'management functions are disabled' and returning empty lists, None, or not_implemented responses. | Test example references non-existent 'test_okta_create_user' and assumes active user management with Okta API mocking, but no create_user method exists in focus and all management methods are disabled without API calls.
- docs/cli/commands/cli-provider.mdx: CLI commands are invoked using 'keep' prefix, not 'cli' | CLI commands are invoked using 'keep' prefix, not 'cli' | CLI commands are invoked using 'keep' prefix, not 'cli' | Lists 'delete' subcommand which has no corresponding implementation in the code
- docs/cli/commands/cli-alert.mdx: CLI commands are invoked using 'keep' prefix, not 'cli' | CLI commands are invoked using 'keep' prefix, not 'cli' | CLI commands are invoked using 'keep' prefix, not 'cli' | Lists 'get' subcommand which has no corresponding implementation in the code
- docs/cli/commands/extractions-list.mdx: Incorrectly describes listing mappings; code function lists extractions | CLI commands are invoked using 'keep' prefix, not 'cli.py'
- docs/cli/commands/extraction-delete.mdx: extraction_id parameter is int type in code, not STRING | CLI commands are invoked using 'keep' prefix, not 'cli.py'
- docs/cli/commands/cli.mdx: CLI root command is invoked as 'keep', not 'cli' | CLI root command is invoked as 'keep', not 'cli' | CLI root command is invoked as 'keep', not 'cli'
- docs/cli/commands/provider-connect.mdx: Incorrect --help description references installation instead of connection; typo 'shuold' instead of 'should'
- docs/cli/commands/mappings-list.mdx: Usage omits 'list' subcommand, contradicting the CLI Help section and function name list_mappings
- docs/cli/commands/cli-workflow.mdx: Command prefix incorrectly listed as 'cli workflow' instead of 'keep workflow' consistent with other CLI docs and project naming | Command prefix incorrectly listed as 'cli workflow' instead of 'keep workflow' consistent with other CLI docs and project naming | Command prefix incorrectly listed as 'cli workflow' instead of 'keep workflow' consistent with other CLI docs and project naming
- docs/cli/commands/workflow-apply.mdx: CLI Help options section incomplete; missing --full-sync and --lookup-by-name parameters present in apply function signature
- docs/cli/commands/provider-list.mdx: Description incorrectly uses singular 'provider' but the command lists multiple available providers
- docs/cli/commands/mappings-delete.mdx: Type documented as STRING but CLI help and code use INTEGER (int); missing (REQUIRED) indicator inconsistent with other docs
- docs/cli/commands/workflow-runs.mdx: CLI command prefix documented as 'cli' but should be 'keep' to match sidebar_title and other command documentation
- docs/deployment/provision/overview.mdx: KEEP_PROVIDERS contains provider configurations only, not deduplication rules (which come from KEEP_DEDUPLICATION_RULES)
- docs/overview/deduplication.mdx: Not every provider has a pre-built partial deduplication rule; only those that define default_fingerprint_fields | Invalid JSON syntax in code example; object keys must be double-quoted strings
- docs/providers/documentation/anthropic-provider.mdx: Documentation incorrectly names the provider as 'Claude Provider' while the code class is 'AnthropicProvider' and other sections consistently use 'Anthropic Provider'
- docs/providers/documentation/console-provider.mdx: Example code omits required first positional argument 'context_manager' for ProvidersFactory.get_provider and incorrectly sets provider_id='mock' for a console provider example
- docs/providers/documentation/ilert-provider.mdx: Internal inconsistency: line 4 refers to 'resolve alerts' while line 10 refers to 'resolve incidents'; standardized to 'incidents' to match the rest of the document and code behavior | Incorrect ilert API endpoint URL; the official ilert events API endpoint is /api/v1/events/{ALERT-SOURCE-API-KEY} without the non-existent /keep/ path segment
- docs/providers/documentation/mock-provider.mdx: Description incorrectly copied from template provider documentation; Mock provider echoes back inputs as per code docstrings.
- docs/providers/documentation/netdata-provider.mdx: Documentation claims Netdata does not support webhooks in on-premises installations, but code implements a specific webhook challenge verification for Netdata.
- docs/providers/documentation/snowflake-provider.mdx: Frontmatter description incorrectly describes the provider as a generic template; it is specifically the Snowflake provider for SQL queries
- docs/providers/documentation/s3-provider.mdx: Code specifies support for 'yaml, json, xml and csv files' only; 'yml' is not listed. | AWS IAM policy version must be '2012-10-17'; '2025-01-15' is invalid.
- docs/providers/documentation/wazuh-provider.mdx: Download instructions reference non-existent 'custom-keep' file (source tree lists only 'custom-keep.py'); adjust to single file and specific chown/chmod targets | Hardcoded cloud-specific URLs and links ('api.keephq.dev', 'platform.keephq.dev') invalid for self-hosted Keep instances
- docs/snippets/providers/auth0-snippet-autogenerated.mdx: The _query method signature includes 'log_type' (required) and 'from_' (optional), but does not include 'previous_users'; documented parameter does not exist in code
- docs/snippets/providers/newrelic-snippet-autogenerated.mdx: The _query method uses the 'nrql' parameter, not 'query'. The example incorrectly lists a non-existent 'query' parameter and mislabels the purpose in the comment.
- docs/snippets/providers/mock-snippet-autogenerated.mdx: The documentation incorrectly states that the mock provider step returns all parameters passed to it, but the code returns only the value of 'command_output' from kwargs if present.
- docs/snippets/providers/posthog-snippet-autogenerated.mdx: Documented default limit for get_session_recording_domains is 100 but code signature shows 500; description incorrectly says 'recordings' instead of 'domains'
- docs/snippets/providers/resend-snippet-autogenerated.mdx: Incorrect workflow example linked for resend provider; bash_example.yml is for bash provider, not resend
- docs/snippets/providers/site24x7-snippet-autogenerated.mdx: Inconsistent lists of valid TLD values: authentication description includes '.au' while valid_tld scope includes '.com.au'. Corrected authentication description to match the scope list and standard Zoho TLDs.
- docs/snippets/providers/sendgrid-snippet-autogenerated.mdx: Example action name incorrectly describes the SendGrid usage as 'Query sendgrid'; the provider's _notify method is for sending emails, not querying data
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Docs Scan Report
• Line 60 Documented CLI usage placeholder for 'condition' parameter is incorrect (''); does not match CLI help section.
SecretManagerFactoryin secretmanagerfactory.py]• Line 16 Factory supported types miss AWS and DB which have dedicated classes and are in the enum [Ref: class
SecretManagerFactoryin secretmanagerfactory.py, classAwsSecretManagerin awssecretmanager.py, classDbSecretManagerin dbsecretmanager.py]• Line 60 Environment variable for tags is SECRET_MANAGER_TAGS in code, not AWS_SECRET_MANAGER_TAGS [Ref: method
AwsSecretManager.__init__in awssecretmanager.py]• Line 97 Kubernetes namespace defaults to 'default' in code, not '.metadata.namespace' [Ref: method
KubernetesSecretManager.__init__in kubernetessecretmanager.py]context_manageras first parameter, but documentation only mentionsprovider_idandProviderConfig[Ref: methodHttpProvider.__init__in http_provider.py]OpensearchserverlessProvider._notifyin opensearchserverless_provider.py]apiin cli.py]OktaIdentityManager.delete_rolein okta_identitymanager.py, methodOktaIdentityManager.create_rolein okta_identitymanager.py, methodOktaIdentityManager.get_rolesin okta_identitymanager.py, methodOktaIdentityManager.get_usersin okta_identitymanager.py, methodOktaIdentityManager.delete_userin okta_identitymanager.py, methodOktaIdentityManager.update_userin okta_identitymanager.py, methodOktaIdentityManager.get_groupsin okta_identitymanager.py]• Lines 159:165 Test example references non-existent 'test_okta_create_user' and assumes active user management with Okta API mocking, but no create_user method exists in focus and all management methods are disabled without API calls. [Ref: method
OktaIdentityManager.get_usersin okta_identitymanager.py, classOktaIdentityManagerin okta_identitymanager.py]providerin cli.py]• Line 9 CLI commands are invoked using 'keep' prefix, not 'cli' [Ref: function
providerin cli.py]• Line 25 CLI commands are invoked using 'keep' prefix, not 'cli' [Ref: function
providerin cli.py]• Lines 32:36 Lists 'delete' subcommand which has no corresponding implementation in the code [Ref: function
connectin cli.py, functionlist_providersin cli.py]alertin cli.py]• Line 9 CLI commands are invoked using 'keep' prefix, not 'cli' [Ref: function
alertin cli.py]• Line 25 CLI commands are invoked using 'keep' prefix, not 'cli' [Ref: function
alertin cli.py]• Lines 33:36 Lists 'get' subcommand which has no corresponding implementation in the code [Ref: function
enrichin cli.py, functionlist_alertsin cli.py]list_extractionin cli.py]• Line 27 CLI commands are invoked using 'keep' prefix, not 'cli.py' [Ref: function
list_extractionin cli.py]delete_extractionin cli.py]• Line 34 CLI commands are invoked using 'keep' prefix, not 'cli.py' [Ref: function
delete_extractionin cli.py]cliin cli.py]• Line 9 CLI root command is invoked as 'keep', not 'cli' [Ref: function
cliin cli.py]• Line 52 CLI root command is invoked as 'keep', not 'cli' [Ref: function
cliin cli.py]connectin cli.py]list_mappingsin cli.py]workflowin cli.py]• Line 9 Command prefix incorrectly listed as 'cli workflow' instead of 'keep workflow' consistent with other CLI docs and project naming [Ref: function
workflowin cli.py]• Line 26 Command prefix incorrectly listed as 'cli workflow' instead of 'keep workflow' consistent with other CLI docs and project naming [Ref: function
workflowin cli.py]applyin cli.py]list_providersin cli.py]delete_mappingin cli.py]workflow_executionsin cli.py]Parser._parse_providers_from_envin parser.py, functionget_deduplication_rules_to_provisionin deduplication_rules_provisioning.py]ProvidersFactory.get_default_deduplication_rulesin providers_factory.py]• Lines 79:82 Invalid JSON syntax in code example; object keys must be double-quoted strings
AnthropicProvider._queryin anthropic_provider.py]ProvidersFactory.get_providerin providers_factory.py, classConsoleProviderin console_provider.py]• Line 24 Incorrect ilert API endpoint URL; the official ilert events API endpoint is /api/v1/events/{ALERT-SOURCE-API-KEY} without the non-existent /keep/ path segment [Ref: method
IlertProvider.__post_ilert_eventin ilert_provider.py]MockProvider._query python::::keep/providers/mock_provider/mock_provider.py::method::MockProvider._notifyin mock_provider.py]webhook_challengein alerts.py]SnowflakeProviderin snowflake_provider.py]S3Provider._queryin s3_provider.py]• Line 19 AWS IAM policy version must be '2012-10-17'; '2025-01-15' is invalid. [Ref: method
S3Provider._queryin s3_provider.py]WazuhProviderin wazuh_provider.py]• Lines 41:45 Hardcoded cloud-specific URLs and links ('api.keephq.dev', 'platform.keephq.dev') invalid for self-hosted Keep instances [Ref: class
WazuhProviderin wazuh_provider.py]Auth0Provider._queryin auth0_provider.py]NewrelicProvider._queryin newrelic_provider.py]MockProvider._queryin mock_provider.py]PosthogProvider.get_session_recording_domainsin posthog_provider.py]ResendProviderin resend_provider.py]Site24X7Provider.__get_headersin site24x7_provider.py]SendgridProvider._notifyin sendgrid_provider.py]Credits
Used: 1
Remaining: 99/100