Duplicate input and/or output query params results in undefined behavior

Spinning this off from #634.

If request.input() param names match request.output() param names in the same request.query(), there is no error thrown, but the result may not be what the user expected.

This bit me when I was trying to do a SELECT and had used the same @name param as a column return value and a bind-argument:

    const name = 'test-user';
    const result = await pool
      .request()
      .input('name', sql.VarChar(255), name)
      .output('id', sql.BigInt)
      .output('name', sql.VarChar(255))
      .output('created_on', sql.DateTime2(3))
      .query(`SELECT TOP 1 [id], [name], [created_on] FROM users WHERE name = @name`);
    if (result.recordsets.length !== 1 || result.recordsets[0].length !== 1) {
      // no results
      return;
    }
    const row = result.recordsets[0][0];
    console.log(`#${row.id}, ${row.name}`);

To fix the issue, I changed input('name', ...) to input('name_query', ...) and @name to @name_query. It may be nice to throw some error here if that happens? Maybe log a warning?

[dhensby]

ah, ok. that’s a bit annoying!

[dhensby]

I would make clear, however, it’s best to properly type your bulk data rather than relying on the library to do it