not work "builtinGlobals" in rule "no-redeclare"

Tell us about your environment

• ESLint Version: v4.0.0
• Node Version: v8.0.0
• npm Version: v5.0.0

Hi, maybe I finded bug. Not wokr rule property “builtinGlobals” in rule “no-redeclare”.

/*eslint no-redeclare: ["error", { "builtinGlobals": true }]*/
/*eslint-env browser*/

var top = 0;
var Object = 0;

console message without error “no-redeclare”

4:1   error  Unexpected var, use let or const instead        no-var
  4:5   error  'top' is assigned a value but never used        no-unused-vars
  4:11  error  Number constants declarations must use 'const'  no-magic-numbers
  5:1   error  Unexpected var, use let or const instead        no-var
  5:5   error  'Object' is assigned a value but never used     no-unused-vars
  5:14  error  Number constants declarations must use 'const'  no-magic-numbers

[mysticatea]

Because you have env: {node: true} in your configuration. The node environment has special function scope which encloses whole code. So variable declarations shadow globals but don’t redeclare globals.

[platinumazure]

Good point @boggddan, we should add a note about module scope for Node and similar environments and recommend no-shadow’s equivalent option. Thanks!