fix(duckdb)!: Remove extra MAP bracket and ARRAY wrap

[VaggelisD]

Fixes #4710

This PR solves two legacy issues:

1. MAPs do not return LISTs anymore but values (1.2 change):

spark3> SELECT MAP('key', 'value')['key'] AS col;
col
value

# Transpilation before this PR
duckdb> SELECT (MAP(['key'], ['value'])['key'])[1] AS col;
┌─────────┐
│   col   │
│ varchar │
├─────────┤
│ v       │
└─────────┘

# Transpilation after this PR
duckdb> SELECT MAP(['key'], ['value'])['key'] AS col;
┌─────────┐
│   col   │
│ varchar │
├─────────┤
│ value   │
└─────────┘

2. Inline indexed LISTs do not need to be wrapped anymore (this was fixed earlier in the year iirc):

spark-sql (default)> SELECT TRY_ELEMENT_AT(ARRAY(1, 2, 3), 2) AS col;
col
2

# Transpilation before this PR
duckdb> SELECT ([1, 2, 3])[2] AS col;
┌───────┐
│  col  │
│ int32 │
├───────┤
│   2   │
└───────┘

# Transpilation after this PR (used to error)
duckdb> SELECT [1, 2, 3][2] AS col;
┌───────┐
│  col  │
│ int32 │
├───────┤
│   2   │
└───────┘

3. A silent T-SQL query turning into command due to typo

Docs

DuckDB 1.2

[tobymao]

you need to add a version to this, i think this is super breaking and breaks compatibility of sqlglot with older versions of duckdb