Skip to content

adding correct insiders name - #1893

Merged
SamMorrowDrums merged 3 commits into
mainfrom
tonytrg/add-correct-insiders
Jan 26, 2026
Merged

adding correct insiders name#1893
SamMorrowDrums merged 3 commits into
mainfrom
tonytrg/add-correct-insiders

Conversation

@tonytrg

@tonytrgtonytrg commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Why

Alignment of names

What changed

MCP impact

  • No tool or API changes
  • Tool schema or behavior changed
  • New tool added

Prompts tested (tool changes only)

Security / limits

  • No security or limits impact
  • Auth / permissions considered
  • Data exposure, filtering, or token/size limits considered

Tool renaming

  • I am renaming tools as part of this PR (e.g. a part of a consolidation effort)
    • I have added the new tool aliases in deprecated_tool_aliases.go
  • I am not renaming tools as part of this PR

Note: if you're renaming tools, you must add the tool aliases. For more information on how to do so, please refer to the official docs.

Lint & tests

  • Linted locally with ./script/lint
  • Tested locally with ./script/test

Docs

  • Not needed
  • Updated (README / docs / examples)

@tonytrg
tonytrg requested a review from a team as a code ownerJanuary 26, 2026 14:45
CopilotAI review requested due to automatic review settings January 26, 2026 14:45

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes a naming inconsistency by renaming InsiderMode to InsidersMode throughout the codebase to align with the actual feature name "insiders mode" and its associated environment variable GITHUB_INSIDERS_MODE and flag --insiders-mode.

Changes:

  • Renamed the InsiderMode field to InsidersMode in the FeatureFlags struct and all configuration structs
  • Updated all references from insider-mode to insiders-mode in flags, viper bindings, and test helpers
  • Updated documentation in README.md and docs/remote-server.md to reflect the corrected naming

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
FileDescription
pkg/github/feature_flags.goRenamed struct field from InsiderMode to InsidersMode
pkg/github/server_test.goUpdated test helper to use insiders-mode key
pkg/github/feature_flags_test.goUpdated test code and variable names to use InsidersMode
internal/ghmcp/server.goUpdated struct fields, comments, and usage of InsidersMode in both MCPServerConfig and StdioServerConfig
internal/ghmcp/server_test.goUpdated test configuration to use InsidersMode
cmd/github-mcp-server/main.goUpdated CLI flag from --insider-mode to --insiders-mode and all viper bindings
README.mdUpdated documentation examples to use --insiders-mode flag and GITHUB_INSIDERS_MODE environment variable
docs/remote-server.mdUpdated documentation reference from GITHUB_INSIDER_MODE to GITHUB_INSIDERS_MODE and --insider-mode to --insiders-mode

@tonytrgtonytrg changed the title adding correct insiders-mode nameadding correct insiders nameJan 26, 2026
Sign up for freeto 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.

3 participants

@tonytrg@SamMorrowDrums