Skip to content

Conversation

@jeongsoolee09
Copy link
Contributor

What This PR Contributes

Future Works

Comment on lines +1224 to +1226
exists(JsonObject jsonObject |
jsonObject = resolveDirectPath(this.getArgument(0).getStringValue())
)

Check warning

Code scanning / CodeQL-Community

Expression can be replaced with a cast

The assignment to [jsonObject](1) in the exists(..) can replaced with an instanceof expression.

class RoutingTargetManifest extends JsonObject {
/** Note: This is NOT its `viewName` property! */
string targetName;

Check notice

Code scanning / CodeQL-Community

Field only used in CharPred

Field is only used in CharPred.
Comment on lines +20 to +23
customController.getAThisNode() = result.getReceiver() and
result.getMethodName() = "getOwnerComponent"
or
exists(TypeTracker t2 | result = getOwnerComponentRef(t2, customController).track(t2, t))

Check warning

Code scanning / CodeQL-Community

Var only used in one side of disjunct.

The [variable t](1) is only used in one side of disjunct.
exists(TypeTracker t2 | result = getOwnerComponentRef(t2, customController).track(t2, t))
}

/* owner component ref */

Check warning

Code scanning / CodeQL-Community

Block comment that is not QLDoc

Block comment could be QLDoc for [the below code](1).
module Test {
private import semmle.javascript.dataflow.TypeTracking

private class ObjFieldStep extends SharedTypeTrackingStep {

Check warning

Code scanning / CodeQL-Community

Dead code

This code is never used, and it's not publicly exported.
}
}

private DataFlow::SourceNode objectWithMethods() {

Check warning

Code scanning / CodeQL-Community

Dead code

This code is never used, and it's not publicly exported.
result.flowsTo(any(DataFlow::CallNode call | call.getCalleeName() = "extend").getAnArgument())
}

private DataFlow::SourceNode getAnAlias(DataFlow::SourceNode object) {

Check warning

Code scanning / CodeQL-Community

Dead code

This code is never used, and it's not publicly exported.
)
}

private predicate methodStepPred(DataFlow::SourceNode object, string name, DataFlow::Node rhs) {

Check warning

Code scanning / CodeQL-Community

Dead code

This code is never used, and it's not publicly exported.
rhs = getAnAlias(object).getAPropertyWrite(name).getRhs()
}

private predicate methodStepSucc(DataFlow::SourceNode object, string name, DataFlow::Node read) {

Check warning

Code scanning / CodeQL-Community

Dead code

This code is never used, and it's not publicly exported.
@data-douser data-douser mentioned this pull request Nov 25, 2025
18 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants