TOOLS-4263 Update the JS -> Go conversion skill based on reviewer comments

[autarch]

No description provided.

[autarch]

• TOOLS-4263 Convert the multi-node dump and restore tests to Go #1143 : 2 dependent PRs (#1144 , #1157 )
• TOOLS-4263 Add multi-node replica set test infrastructure #1142
• TOOLS-4263 Convert the system-CA TLS test to Go #1141
• TOOLS-4263 Convert cluster-topology tests to Go #1140
• TOOLS-4263 Convert mongofiles get and search tests to Go #1139
• TOOLS-4263 Convert mongofiles put and storage-option tests to Go #1138
• TOOLS-4263 Drop the mopt alias for the driver's options package #1137
• TOOLS-4263 Convert mongotop tests to Go #1136
• TOOLS-4263 Convert mongostat tests to Go #1135
• TOOLS-4263 Convert mongoimport option validation tests to Go #1134
• TOOLS-4263 Convert typed-column and parseGrace import tests to Go #1133
• TOOLS-4263 Convert export/import BSON type edge tests to Go #1132
• TOOLS-4263 Convert CSV and TSV round trip tests to Go #1131
• TOOLS-4263 Convert mongodump auth profile dump test to Go #1130
• TOOLS-4263 Convert mongodump views round-trip tests to Go #1129
• TOOLS-4263 Convert mongodump oplog consistency tests to Go #1104
• TOOLS-4263 Convert mongodump query and data behavior tests to Go #1102
• TOOLS-4263 Convert mongodump flag validation tests to Go #1098
• TOOLS-4263 Convert mongorestore users and roles temp collection tests to Go #1097
• TOOLS-4263 Convert mongorestore drop, path and stdin tests to Go #1096
• TOOLS-4263 Convert mongorestore namespace mapping tests to Go #1093
• TOOLS-4263 Convert mongorestore bulk sizing tests to Go #1091
• TOOLS-4263 Convert mongorestore index round-trip tests to Go #1074
• TOOLS-4263 Convert mongorestore collection-options and metadata tests to Go #1073
• TOOLS-4263 Update the JS -> Go conversion skill based on reviewer comments #1148 👈 (View in Graphite)
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

[Copilot]

Pull request overview

Updates the mongo-tools-js-to-go conversion Skill guidance to incorporate recurring reviewer feedback, with the goal of producing more reviewable, idiomatic Go integration tests when translating JS/resmoke tests in mongo-tools.

Changes:

• Expands Go test code conventions (helper placement, bson guidance, loops, and set usage).
• Adds new guidance sections on naming/scope, avoiding JS-provenance commentary in Go code, and maintaining fidelity to original JS test behavior.
• Tightens the conversion process checklist to explicitly call out coverage gaps, unnecessary additions, and PR size management.

Suppressed comments (1)

.claude/skills/mongo-tools-js-to-go/SKILL.md:134

• “the commit message, which is also the PR description” is not generally true on GitHub and could be misleading. If the intent is “put provenance in the commit message and the PR description”, it’s clearer to say so explicitly.

Provenance belongs in the **commit message**, which is also the PR description: name the JS
files the commit converts and deletes there.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.