id-match onlyDeclarations evaluates property assignment value (when it should not)

What version of ESLint are you using?

v2.8.0

What parser (default, Babel-ESLint, etc.) are you using?

babel-eslint

Please show your full configuration:

{
    "rules": {
        "id-match": [
            2,
            "(^[A-Za-z]+(?:[A-Z][a-z]*)*\\d*$)|(^[A-Z]+(_[A-Z]+)*(_\\d$)*$)|(^(_|\\$)$)",
            {
                "onlyDeclarations": true,
                "properties": true
            }
        ]
    }
}

What did you do? Please include the actual source code causing the issue.

The subject code:

export default {
    context: __dirname
};

What did you expect to happen?

I expected no linting error.

What actually happened? Please include the actual, raw output from ESLint.

I got a linting error:

║ Line     │ Column   │ Type     │ Message                                                │ Rule ID              ║
╟──────────┼──────────┼──────────┼────────────────────────────────────────────────────────┼──────────────────────╢
║ 25       │ 14       │ error    │ Identifier '__dirname' does not match the pattern      │ id-match             ║
║          │          │          │ '(^[A-Za-z]+(?:[A-Z][a-z]*)*\d*$)|(^[A-Z]+(_[A-        │                      ║
║          │          │          │ Z]+)*(_\d$)*$)|(^(_|\$)$)'                             │                      ║

This is in relation to the following PR, https://github.com/eslint/eslint/pull/5631.

[platinumazure]

I think it should be keys only. If someone wanted to flag property values, too, they could turn off onlyDeclarations so it flags uses of identifiers that are not declared in the same script or module.

[mikesherov]

@kaicataldo care to review https://github.com/eslint/eslint/pull/5960 ?