Skip to content

feat(bigquery): add routines introspection - #8

Merged
vistart merged 4 commits into
release/v1.0.0.dev1from
feature/introspection-coverage
Nov 9, 2026
Merged

vistart merged 4 commits into
release/v1.0.0.dev1from
feature/introspection-coverage

Conversation

@vistart

@vistart vistart commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Description

Adds the missing bigquery introspection coverage and aligns the CLI surface with what the backend can actually deliver.

  • routines from INFORMATION_SCHEMA.ROUTINES
  • documented engine absences: types, sequences, operators, reservations/jobs

The ddl introspection type is no longer advertised for this backend because
supports_ddl_extraction() is permanently False. introspect ddl is now an
argparse usage error (exit 2); the backend DDL expression/formatter remains for
a future implementation.

Adds # (no linked issue; operator-directed introspection coverage work)

Type of change

  • feat: A new feature
  • docs: Documentation only changes
  • test: Adding missing tests or correcting existing tests
  • ci: Changes to CI configuration files and scripts
  • perf
  • chore
  • build
  • revert

Breaking Change

  • Yes
  • No

The removed ddl CLI type never produced an answer for an ordinary table, so
its removal changes the help/type surface rather than a working capability.

Applicable rhosocial-activerecord Version Range:

>=1.0.0.dev30,<2.0.0

Related Repositories/Packages

  • rhosocial-activerecord core: release/v1.0.0.dev30
  • rhosocial-activerecord-testsuite: release/v1.0.0.dev17
  • This backend's own release branch: release/v1.0.0.dev1

Testing

  • CI run 42591258426: success (all jobs)
  • Local offline tests: 122 offline introspection/CLI tests
  • introspect --help no longer lists ddl
  • introspect ddl exits 2 with invalid choice

Test Plan:

The CI run above is the full matrix. Targeted offline suites cover the new
formatters, parsers, sync/async dispatch, capability gates and CLI vocabulary.

Checklist

  • My code follows the project's code style guidelines.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.
  • I have added a Changelog fragment (changelog.d/<pr_number>.<type>.md).
    Not added: the operator explicitly requested no changelog fragments.
  • I have verified that my changes do not introduce SQL injection vulnerabilities.
  • I have checked for potential performance regressions.

Additional Notes

The backend capability/formatter for DDL extraction is intentionally left in
place; only the CLI type was removed until a real implementation exists.

Branch Commits

22063e6 refactor(bigquery): stop advertising ddl introspection
7fb9260 ci(bigquery): test against testsuite release/v1.0.0.dev17
c7a4ac5 feat(bigquery): add routines introspection

@codecov

codecov Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.11111% with 8 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.11%. Comparing base (693437a) to head (9954eff).

Files with missing lines Patch % Lines
...ackend/impl/bigquery/introspection/introspector.py 82.60% 4 Missing and 4 partials ⚠️
Additional details and impacted files
@@                   Coverage Diff                   @@
##           release/v1.0.0.dev1       #8      +/-   ##
=======================================================
+ Coverage                74.65%   75.11%   +0.46%     
=======================================================
  Files                       44       44              
  Lines                     3129     3203      +74     
  Branches                   412      419       +7     
=======================================================
+ Hits                      2336     2406      +70     
- Misses                     707      708       +1     
- Partials                    86       89       +3     
Flag Coverage Δ
python-3.12 75.11% <91.11%> (+0.46%) ⬆️
unittests 75.11% <91.11%> (+0.46%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sonarqubecloud

sonarqubecloud Bot commented Nov 9, 2026

Copy link
Copy Markdown

@vistart vistart added the enhancement New feature or request label Nov 9, 2026
@vistart vistart self-assigned this Nov 9, 2026
@vistart
vistart merged commit 2b84cde into release/v1.0.0.dev1 Nov 9, 2026
11 checks passed
@vistart
vistart deleted the feature/introspection-coverage branch November 9, 2026 17:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant