Skip to content

feat(snowflake): add routines, sequences and catalog-object introspection - #10

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

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

Conversation

@vistart

@vistart vistart commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Description

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

  • routines from FUNCTIONS and PROCEDURES
  • standalone sequences
  • stages, file formats, pipes, tasks and streams
  • identity and virtual-column facts on columns

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.dev2

Testing

  • CI run 42592054261: success (all jobs)
  • Local offline tests: 135 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

da737a1 refactor(snowflake): stop advertising ddl introspection
b2ea3d9 ci(snowflake): test against testsuite release/v1.0.0.dev17
6cd1968 feat(snowflake): add routines, sequences and catalog-object introspection

@codecov

codecov Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 92.77778% with 26 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.71%. Comparing base (edb5cda) to head (e518a2f).

Files with missing lines Patch % Lines
...ckend/impl/snowflake/introspection/introspector.py 87.73% 13 Missing and 13 partials ⚠️
Additional details and impacted files
@@                   Coverage Diff                   @@
##           release/v1.0.0.dev2      #10      +/-   ##
=======================================================
+ Coverage                80.96%   81.71%   +0.75%     
=======================================================
  Files                      117      117              
  Lines                     5206     5546     +340     
  Branches                   768      804      +36     
=======================================================
+ Hits                      4215     4532     +317     
- Misses                     760      771      +11     
- Partials                   231      243      +12     
Flag Coverage Δ
python-3.12 81.71% <92.77%> (+0.75%) ⬆️
unittests 81.71% <92.77%> (+0.75%) ⬆️

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.

@vistart vistart added the enhancement New feature or request label Nov 9, 2026
@vistart vistart self-assigned this Nov 9, 2026
@sonarqubecloud

sonarqubecloud Bot commented Nov 9, 2026

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@vistart
vistart merged commit c6cfb4f into release/v1.0.0.dev2 Nov 9, 2026
11 of 12 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