Skip to content

Add lkr-project-helper skill, model auto-config on push, and create_developer_copy SDK method - #47

Merged
bwebs merged 2 commits into
mainfrom
bryan/new-project-skill
Sep 15, 2026
Merged

bwebs merged 2 commits into
mainfrom
bryan/new-project-skill

Conversation

@bwebs

@bwebs bwebs commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • New Skill (lkr-project-helper): Adds installable skill for creating, configuring, generating, synchronizing, and deploying LookML projects via uvx lkr-dev-cli, lkr code-mode sandbox --dev-mode, and generate_lookml_with_new_files.
    • Supports interactive project instantiation from database schemas/tables, raw SQL (bq_view_lookml_generator.py), and natural language intent.
    • Accompanied by modular --dev-mode Python helper scripts for connection introspection, model binding, LookML generation, bare Git setup, asset uploads, and LookML dashboard-to-UDD conversion.
  • LookML Push (lkr tools lookml push):
    • Added --reset/--no-reset flag (default True) to reset developer workspace to production prior to pushing files.
    • Automatically creates or updates LookML model configurations when .model.lkml files are pushed with declared connection: names.
  • Extended SDK:
    • Added create_developer_copy (and developer_copy alias) to ExtendedLooker40SDK for POST /projects/{project_id}/developer_copy.
  • Code Mode Safeguards:
    • Blocks update_session by default to prevent unintended mutations of shared OAuth tokens across CLI executions, with --allow-update-session available for explicit override.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces a new Looker Project Helper skill (lkr-project-helper) along with a suite of helper scripts to automate LookML project creation, database connection verification, model configuration, validation, and deployment. It also updates the CLI push command to support workspace resetting and automatic model configuration, adds a new create_developer_copy method to the extended SDK, and implements safety checks in code-mode to block or warn about update_session calls. Feedback on the changes highlights several potential runtime errors in the newly added scripts due to accessing properties on Looker SDK objects as if they were dictionaries instead of class instances, as well as a recurring typo in the # type: ignore comments.

Comment thread skills/lkr-project-helper/scripts/bq_view_lookml_generator.py
Comment thread skills/lkr-project-helper/scripts/ensure_project_and_model.py
Comment thread skills/lkr-project-helper/scripts/init_project.py
Comment thread skills/lkr-project-helper/scripts/init_project.py
Comment thread skills/lkr-project-helper/scripts/bq_view_lookml_generator.py
@bwebs
bwebs merged commit 40000a1 into main Sep 15, 2026
1 check passed
@bwebs
bwebs deleted the bryan/new-project-skill branch September 15, 2026 19:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant