Skip to content

Remove GraphQL AppSec contrib from Steep ignored files#5356

Open
y9v wants to merge 2 commits intomasterfrom
appsec-daily-rbs-chore-12-feb
Open

Remove GraphQL AppSec contrib from Steep ignored files#5356
y9v wants to merge 2 commits intomasterfrom
appsec-daily-rbs-chore-12-feb

Conversation

@y9v
Copy link
Member

@y9v y9v commented Feb 12, 2026

What does this PR do?
GraphQL contrib was in the list of files that Steep ignores. This PR removes those files from the ignore list, and fixes type errors.

Motivation:
We don't want any AppSec files to be ignored by Steep.

Change log entry
None.

Additional Notes:
APPSEC-60626

How to test the change?
CI

@y9v y9v self-assigned this Feb 12, 2026
@y9v y9v requested review from a team as code owners February 12, 2026 16:55
@github-actions github-actions bot added integrations Involves tracing integrations appsec Application Security monitoring product labels Feb 12, 2026
@github-actions
Copy link

github-actions bot commented Feb 12, 2026

Typing analysis

Ignored files

This PR clears 5 ignored files. It increases the percentage of typed files from 42.98% to 43.54% (+0.56%).

Ignored files (+0-5)Cleared:
lib/datadog/appsec/contrib/graphql/appsec_trace.rb
lib/datadog/appsec/contrib/graphql/gateway/multiplex.rb
lib/datadog/appsec/contrib/graphql/gateway/watcher.rb
lib/datadog/appsec/contrib/graphql/integration.rb
lib/datadog/appsec/contrib/graphql/patcher.rb

@pr-commenter
Copy link

pr-commenter bot commented Feb 12, 2026

Benchmarks

Benchmark execution time: 2026-02-12 17:37:16

Comparing candidate commit 3c7a780 in PR branch appsec-daily-rbs-chore-12-feb with baseline commit c2bfb0b in branch master.

Found 1 performance improvements and 1 performance regressions! Performance is the same for 42 metrics, 2 unstable metrics.

scenario:error - error tracking, with_error=true - all

  • 🟥 throughput [-401.423op/s; -378.576op/s] or [-5.654%; -5.332%]

scenario:tracing - Propagation - Trace Context

  • 🟩 throughput [+3255.751op/s; +3353.148op/s] or [+9.540%; +9.825%]

@y9v y9v force-pushed the appsec-daily-rbs-chore-12-feb branch from 3c7a780 to e0c05fe Compare February 13, 2026 13:47
@datadog-datadog-prod-us1
Copy link
Contributor

datadog-datadog-prod-us1 bot commented Feb 13, 2026

✅ Tests

🎉 All green!

❄️ No new flaky tests detected
🧪 All tests passed

🎯 Code Coverage (details)
Patch Coverage: 100.00%
Overall Coverage: 95.10% (+0.00%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 0b2cbc5 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@y9v y9v force-pushed the appsec-daily-rbs-chore-12-feb branch from e0c05fe to 0b2cbc5 Compare February 13, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

appsec Application Security monitoring product integrations Involves tracing integrations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants