FIX: Address Copilot review feedback

- Fix architecture detection reference: ddbc_bindings.py not __init__.py
- Fix stored procedure syntax: use positional params without named param mixing

Author: dlevy-msft-sql

.github/copilot-instructions.md

@@ -229,7 +229,7 @@ The CI system tests:
 
 ## Architecture Detection and Loading
 
-The `__init__.py` module implements sophisticated architecture detection:
+The `ddbc_bindings.py` module implements sophisticated architecture detection:
 - **Windows**: Normalizes `win64/amd64/x64` → `x64`, `win32/x86` → `x86`, `arm64` → `arm64`
 - **macOS**: Runtime architecture detection, always loads from universal2 binary
 - **Linux**: Maps `x64/amd64` → `x86_64`, `arm64/aarch64` → `arm64`

llms.txt

@@ -178,7 +178,7 @@ for column in cursor.description:
 
 ```python
 # Call a stored procedure
-cursor.execute("EXEC GetUserById @UserId = ?", (user_id,))
+cursor.execute("EXEC GetUserById ?", (user_id,))
 result = cursor.fetchone()
 
 # Stored procedure with output