`describe_table` should describe tables outside the data source in Python
Maintainers usually reply within 2 days
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 1-2 days
- Newbie friendliness
- 35/100
Research direction
Start with _describe_table_text() on the Python side, which currently raises when a table is missing from source.table_ids, and read the R change in #470 to see the intended behavior. The new cases belong in tests/shared/describe-table.json so both suites check the same results. Done means tables outside the data source are described with a no-documentation note, trusted-only mode still refuses them, and alternate names like main.sales resolve to the table's label.
Written by the indexing model from the issue text.
Description
Same thing as #469, but for the Python side.
In the R package, #470 lets describe_table describe any table the connection can query, not just the data source's tables, so it agrees with run_sql. Python's _describe_table_text() still raises when a table isn't in source.table_ids, so the agent can query a table it found with run_sql but can't describe it.
To match R:
- Describe tables outside the data source, with a note that no documentation covers them. On Snowflake and Databricks, run the usual access check first.
- Keep refusing them in trusted-only mode.
- Resolve alternate names for data source tables (
main.salesforsales) to the table's label, so its documentation still shows.
Pin these in tests/shared/describe-table.json so both suites test the same behavior.
- Dominant language
- Python
- Stars
- 69
- Forks
- 5
- Avg merge
- 5d 3h
- Merged PRs (30d)
- 33
Getting set up
This project ships no dev container, Dockerfile or contributing guide, so setting up is up to you: start from its README, and see our first-contribution guide for the general steps.
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from posit-dev/commons
-
Remove the `pull_request_target` workflow before GitHub blocks itPossibly taken @jat255 claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 67/100
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 80/100
Maintainers usually reply within 2 days
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 2 days
All issues in posit-dev/commons
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
aws-samples/sample-per-model-bedrock#795 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Maintainers usually reply within 1 day
-
Pretalx reports older version to be the latest release than what is actually installed and availableOpen
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 70/100
Maintainers usually reply within 1 day
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
micropython/micropython#19906 ·
Maintainers usually reply within 1 day