fix(skill): use package data for skill install + docs updates

Version 0.1.1

Fixes:
- skill install now works when installed via pip by using
  importlib.resources to read SKILL.md from package data
- Moved skills/notebooklm/SKILL.md to src/notebooklm/data/SKILL.md

Documentation updates:
- Added missing source wait and artifact wait commands to CLI reference
- Updated export methods to show ExportType enum instead of integers
- Fixed slide-deck download docs (PDF, not PNGs)
- Fixed incorrect --wait usage pattern in getting-started.md

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: teng-lin

docs/cli-reference.md

@@ -75,6 +75,7 @@ See [Configuration](configuration.md) for details on environment variables and C
 | `rename <id> <title>` | Source ID, new title | - | `source rename src123 "New Name"` |
 | `refresh <id>` | Source ID | - | `source refresh src123` |
 | `delete <id>` | Source ID | - | `source delete src123` |
+| `wait <id>` | Source ID | `--timeout`, `--interval` | `source wait src123` |
 
 ### Research Commands (`notebooklm research <cmd>`)
 
@@ -105,8 +106,9 @@ See [Configuration](configuration.md) for details on environment variables and C
 | `get <id>` | Artifact ID | - | `artifact get art123` |
 | `rename <id> <title>` | Artifact ID, title | - | `artifact rename art123 "Title"` |
 | `delete <id>` | Artifact ID | - | `artifact delete art123` |
-| `export <id>` | Artifact ID | - | `artifact export art123` |
+| `export <id>` | Artifact ID | `--type [docs|sheets]`, `--title` | `artifact export art123 --type sheets` |
 | `poll <task_id>` | Task ID | - | `artifact poll task123` |
+| `wait <id>` | Artifact ID | `--timeout`, `--interval` | `artifact wait art123` |
 | `suggestions` | - | - | `artifact suggestions` |
 
 ### Download Commands (`notebooklm download <type>`)
@@ -307,14 +309,14 @@ notebooklm generate audio [description] [OPTIONS]
 
 **Examples:**
 ```bash
-# Basic podcast
+# Basic podcast (starts async, returns immediately)
 notebooklm generate audio
 
 # Debate format with custom instructions
 notebooklm generate audio "Compare the two main viewpoints" --format debate
 
-# Wait for completion
-notebooklm generate audio --wait
+# Generate and wait for completion
+notebooklm generate audio "Focus on key points" --wait
 ```
 
 ### Generate: `video`

docs/getting-started.md

@@ -118,14 +118,10 @@ notebooklm ask "What are the key themes in this article?"
 ### Step 5: Generate a Podcast
 
 ```bash
-notebooklm generate audio "Focus on the history and future predictions"
+notebooklm generate audio "Focus on the history and future predictions" --wait
 ```
 
-This starts an async generation job. Wait for it:
-
-```bash
-notebooklm generate audio --wait
-```
+This generates a podcast and waits for completion (takes 2-5 minutes).
 
 ### Step 6: Download the Result
 

docs/python-api.md

@@ -284,7 +284,7 @@ await client.sources.refresh(nb_id, src.id)  # Re-fetch URL content
 | `download_audio(notebook_id, output_path, artifact_id=None)` | `str, str, str` | `str` | Download audio to file (MP4/MP3) |
 | `download_video(notebook_id, output_path, artifact_id=None)` | `str, str, str` | `str` | Download video to file (MP4) |
 | `download_infographic(notebook_id, output_path, artifact_id=None)` | `str, str, str` | `str` | Download infographic to file (PNG) |
-| `download_slide_deck(notebook_id, output_dir, artifact_id=None)` | `str, str, str` | `list[str]` | Download slides to directory (PNGs) |
+| `download_slide_deck(notebook_id, output_path, artifact_id=None)` | `str, str, str` | `str` | Download slide deck as PDF |
 
 **Download Methods:**
 
@@ -301,15 +301,14 @@ path = await client.artifacts.download_video(nb_id, "video.mp4")
 # Download infographic
 path = await client.artifacts.download_infographic(nb_id, "infographic.png")
 
-# Download slide deck (creates multiple files)
-slide_paths = await client.artifacts.download_slide_deck(nb_id, "./slides/")
-# Returns: ["./slides/slide_001.png", "./slides/slide_002.png", ...]
+# Download slide deck as PDF
+path = await client.artifacts.download_slide_deck(nb_id, "./slides.pdf")
+# Returns: "./slides.pdf"
 ```
 
 **Notes:**
 - If `artifact_id` is not specified, downloads the first completed artifact of that type
 - Raises `ValueError` if no completed artifact is found
-- `download_slide_deck` creates the output directory if it doesn't exist
 - Some URLs require browser-based download (handled automatically)
 
 #### Export Methods
@@ -318,20 +317,23 @@ Export artifacts to Google Docs or Google Sheets.
 
 | Method | Parameters | Returns | Description |
 |--------|------------|---------|-------------|
-| `export_report(notebook_id, artifact_id, title="Export", export_type=1)` | `str, str, str, int` | `Any` | Export report to Google Docs |
-| `export_data_table(notebook_id, artifact_id, title="Export")` | `str, str, str` | `Any` | Export data table to Google Sheets |
-| `export(notebook_id, artifact_id=None, content=None, title="Export", export_type=1)` | `str, str, str, str, int` | `Any` | Generic export to Docs/Sheets |
+| `export_report(notebook_id, artifact_id, title, export_type)` | `str, str, str, ExportType` | `Any` | Export report to Google Docs/Sheets |
+| `export_data_table(notebook_id, artifact_id, title)` | `str, str, str` | `Any` | Export data table to Google Sheets |
+| `export(notebook_id, artifact_id, content, title, export_type)` | `str, str, str, str, ExportType` | `Any` | Generic export to Docs/Sheets |
 
-**Export Types:**
-- `export_type=1`: Export to Google Docs
-- `export_type=2`: Export to Google Sheets
+**Export Types (ExportType enum):**
+- `ExportType.DOCS` (1): Export to Google Docs
+- `ExportType.SHEETS` (2): Export to Google Sheets
 
 ```python
+from notebooklm import ExportType
+
 # Export a report to Google Docs
 result = await client.artifacts.export_report(
     nb_id,
     artifact_id="report_123",
-    title="My Briefing Doc"
+    title="My Briefing Doc",
+    export_type=ExportType.DOCS
 )
 # result contains the Google Docs URL
 
@@ -343,12 +345,12 @@ result = await client.artifacts.export_data_table(
 )
 # result contains the Google Sheets URL
 
-# Generic export (e.g., export any artifact to Docs)
+# Generic export (e.g., export any artifact to Sheets)
 result = await client.artifacts.export(
     nb_id,
     artifact_id="artifact_789",
     title="Exported Content",
-    export_type=1  # 1=Docs, 2=Sheets
+    export_type=ExportType.SHEETS
 )
 ```
 
@@ -694,6 +696,14 @@ class SlideDeckLength(Enum):
     SHORT = 2
 ```
 
+### Export
+
+```python
+class ExportType(Enum):
+    DOCS = 1    # Export to Google Docs
+    SHEETS = 2  # Export to Google Sheets
+```
+
 ### Chat Configuration
 
 ```python

docs/stability.md

@@ -67,7 +67,7 @@ SourceError, SourceProcessingError, SourceTimeoutError, SourceNotFoundError
 RPCError
 
 # Enums
-AudioFormat, VideoFormat, StudioContentType, etc.
+AudioFormat, VideoFormat, StudioContentType, ExportType, etc.
 
 # Auth
 AuthTokens, DEFAULT_STORAGE_PATH

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "notebooklm-py"
-version = "0.1.0"
+version = "0.1.1"
 description = "Unofficial Python client for Google NotebookLM API"
 readme = "README.md"
 requires-python = ">=3.9"

src/notebooklm/__init__.py

@@ -13,7 +13,7 @@
     See docs/troubleshooting.md for guidance on handling API changes.
 """
 
-__version__ = "0.1.0"
+__version__ = "0.1.1"
 
 # Public API: Authentication
 from .auth import AuthTokens, DEFAULT_STORAGE_PATH

src/notebooklm/cli/skill.py

@@ -4,19 +4,29 @@
 """
 
 import re
+from importlib import resources
 from pathlib import Path
+from typing import Optional
 
 import click
 
 from .helpers import console
 
 
 # Skill paths
-SKILL_SOURCE = Path(__file__).parent.parent.parent.parent / "skills" / "notebooklm" / "SKILL.md"
 SKILL_DEST_DIR = Path.home() / ".claude" / "skills" / "notebooklm"
 SKILL_DEST = SKILL_DEST_DIR / "SKILL.md"
 
 
+def get_skill_source_content() -> Optional[str]:
+    """Read the skill source file from package data."""
+    try:
+        # Python 3.9+ way to read package data
+        return resources.files("notebooklm").joinpath("data", "SKILL.md").read_text()
+    except (FileNotFoundError, TypeError):
+        return None
+
+
 def get_package_version() -> str:
     """Get the current package version."""
     try:
@@ -51,19 +61,17 @@ def install():
     Copies the skill file to ~/.claude/skills/notebooklm/SKILL.md
     and embeds the current package version for tracking.
     """
-    # Check if source exists
-    if not SKILL_SOURCE.exists():
-        console.print(f"[red]Error:[/red] Skill source not found at {SKILL_SOURCE}")
-        console.print("This may indicate an incomplete installation.")
+    # Read skill content from package data
+    content = get_skill_source_content()
+    if content is None:
+        console.print("[red]Error:[/red] Skill source not found in package data.")
+        console.print("This may indicate an incomplete or corrupted installation.")
+        console.print("Try reinstalling: pip install --force-reinstall notebooklm-py")
         raise SystemExit(1)
 
     # Create destination directory
     SKILL_DEST_DIR.mkdir(parents=True, exist_ok=True)
 
-    # Read skill content
-    with open(SKILL_SOURCE) as f:
-        content = f.read()
-
     # Embed version in skill file (after frontmatter)
     version = get_package_version()
     version_comment = f"<!-- notebooklm-py v{version} -->\n"

skills/notebooklm/SKILL.md src/notebooklm/data/SKILL.mdskills/notebooklm/SKILL.md renamed to src/notebooklm/data/SKILL.md

@@ -1,8 +1,7 @@
 """Tests for skill CLI commands."""
 
 import pytest
-from pathlib import Path
-from unittest.mock import patch, mock_open
+from unittest.mock import patch
 from click.testing import CliRunner
 
 from notebooklm.notebooklm_cli import cli
@@ -19,26 +18,25 @@ class TestSkillInstall:
     def test_skill_install_creates_directory_and_file(self, runner, tmp_path):
         """Test that install creates the skill file."""
         skill_dest = tmp_path / "skills" / "notebooklm" / "SKILL.md"
+        mock_source_content = "---\nname: notebooklm\n---\n# Test"
 
         with patch("notebooklm.cli.skill.SKILL_DEST", skill_dest), \
              patch("notebooklm.cli.skill.SKILL_DEST_DIR", skill_dest.parent), \
-             patch("notebooklm.cli.skill.SKILL_SOURCE") as mock_source:
+             patch("notebooklm.cli.skill.get_skill_source_content", return_value=mock_source_content):
 
-            # Create a mock source file
-            mock_source.exists.return_value = True
-            mock_source_content = "---\nname: notebooklm\n---\n# Test"
-
-            with patch("builtins.open", mock_open(read_data=mock_source_content)):
-                result = runner.invoke(cli, ["skill", "install"])
+            result = runner.invoke(cli, ["skill", "install"])
 
-            # Check command succeeded (may fail due to mock complexity, but structure is right)
-            assert "install" in result.output.lower() or result.exit_code == 0
+            assert result.exit_code == 0
+            assert "installed" in result.output.lower()
+            assert skill_dest.exists()
 
     def test_skill_install_source_not_found(self, runner, tmp_path):
         """Test error when source file doesn't exist."""
-        skill_source = tmp_path / "nonexistent" / "SKILL.md"
+        skill_dest = tmp_path / "skills" / "notebooklm" / "SKILL.md"
 
-        with patch("notebooklm.cli.skill.SKILL_SOURCE", skill_source):
+        with patch("notebooklm.cli.skill.SKILL_DEST", skill_dest), \
+             patch("notebooklm.cli.skill.SKILL_DEST_DIR", skill_dest.parent), \
+             patch("notebooklm.cli.skill.get_skill_source_content", return_value=None):
             result = runner.invoke(cli, ["skill", "install"])
 
         assert result.exit_code == 1