Skip to content

(feat): add app name to mongo atlas query#348

Open
abnegate wants to merge 1 commit intomainfrom
feat-add-appname
Open

(feat): add app name to mongo atlas query#348
abnegate wants to merge 1 commit intomainfrom
feat-add-appname

Conversation

@abnegate
Copy link
Copy Markdown
Member

What does this PR do?

(Provide a description of what this PR does.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

(Write your answer here.)

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Apr 30, 2026

Greptile Summary

This PR adds an appName to the MongoClient options in the Node.js Mongo Atlas template so connection activity can be identified in Atlas monitoring. However, the value 'devrel-integration-atlas-appwrite-php' is incorrect for this Node.js file — it appears to have been copied from a PHP template and the runtime suffix was never updated.

Confidence Score: 3/5

Not safe to merge as-is — the appName misidentifies the runtime in Atlas monitoring.

Single P1 finding: the appName value ends with -php in a Node.js file, causing all Atlas telemetry for this integration to be misattributed. The fix is a one-line string change.

node/query-mongo-atlas/src/mongo.js

Important Files Changed

Filename Overview
node/query-mongo-atlas/src/mongo.js Adds appName to MongoClient options, but the value incorrectly ends with -php instead of -node.

Reviews (1): Last reviewed commit: "(feat): add app name to mongo atlas quer..." | Re-trigger Greptile

Comment thread node/query-mongo-atlas/src/mongo.js
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.

1 participant