Use ALL_TAB_COLUMNS that does not show hidden columns by default

[yahonda]

Summary

• Use ALL_TAB_COLUMNS instead of ALL_TAB_COLS in get_field_definitions to exclude hidden columns (e.g., those created by functional indexes)
• ALL_TAB_COLUMNS excludes hidden columns by default and is available in Oracle 11g and newer
  • Oracle 11g - ALL_TAB_COLUMNS
  • Oracle 23c - ALL_TAB_COLUMNS

Based on #208

Test plan

• Verify test workflow passes (Oracle 23c Free)
• Verify test_11g workflow passes (Oracle 11g XE)
• New test creates a table with a functional index (which generates a hidden column) and verifies hidden columns are excluded from field definitions

Co-Authored-By: Jochen Schug 4573581+joschug@users.noreply.github.com

Generated with Claude Code

[Copilot]

Pull request overview

This PR updates how %ROWTYPE field definitions are derived for table/view-backed record types by switching the data dictionary view used to read columns, with a new spec to ensure hidden columns created by functional indexes are excluded.

Changes:

• Replace ALL_TAB_COLS with ALL_TAB_COLUMNS in get_field_definitions for table/view column discovery.
• Add a regression spec that creates a functional index (which introduces a hidden column) and asserts hidden columns do not appear in %ROWTYPE fields.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
lib/plsql/procedure.rb	Uses ALL_TAB_COLUMNS when building field metadata for TABLE/VIEW %ROWTYPE records to avoid hidden columns.
spec/plsql/procedure_spec.rb	Adds a regression test ensuring hidden columns (e.g., SYS_NC...) are excluded from %ROWTYPE field definitions.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.