events: refactor [customInspectSymbol]

[himself65]

No description provided.

[benjamingr]

The CI failures look real since the []s were removed, is that intentional?

[himself65]

Path: parallel/test-eventtarget-memoryleakwarning
Error: --- stderr ---
node:assert:122
  throw new AssertionError(obj);
  ^

AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
+ actual - expected

+ 'Possible EventTarget memory leak detected. 3 foo listeners added to AbortSignal. Use events.setMaxListeners() to increase limit'
- 'Possible EventTarget memory leak detected. 3 foo listeners added to [AbortSignal]. Use events.setMaxListeners() to increase limit'
    at Object.<anonymous> (/home/runner/work/node/node/test/common/index.js:517:14)
    at Object.MaxListenersExceededWarning (/home/runner/work/node/node/test/common/index.js:381:15)
    at process.<anonymous> (/home/runner/work/node/node/test/common/index.js:541:33)
    at process.emit (node:events:369:20)
    at doEmitWarning (node:internal/process/warning:70:11)
    at processTicksAndRejections (node:internal/process/task_queues:80:21) {
  generatedMessage: true,
  code: 'ERR_ASSERTION',
  actual: 'Possible EventTarget memory leak detected. 3 foo listeners added to AbortSignal. Use events.setMaxListeners() to increase limit',
  expected: 'Possible EventTarget memory leak detected. 3 foo listeners added to [AbortSignal]. Use events.setMaxListeners() to increase limit',
  operator: 'strictEqual'
}

[benjamingr]

Appreciative of the work and involvement.

blocking for now just so this doesn't land with the breaking change before we've discussed why the breaking change is required or is a good idea.