Skip to content

Commit b01e740

Browse files
teng-linclaude
andcommitted
ci: remove debug code after fixing report download test
The test now passes with the correct notebook ID configured. Keep test_filter input for running specific tests on demand. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
1 parent bf5c8f2 commit b01e740

File tree

3 files changed

+3
-77
lines changed

3 files changed

+3
-77
lines changed

.github/workflows/nightly.yml

Lines changed: 0 additions & 67 deletions
Original file line numberDiff line numberDiff line change
@@ -61,73 +61,6 @@ jobs:
6161
if: runner.os == 'Linux'
6262
run: playwright install-deps
6363

64-
- name: Debug - Print notebook IDs
65-
shell: bash
66-
env:
67-
NOTEBOOKLM_READ_ONLY_NOTEBOOK_ID: ${{ secrets.NOTEBOOKLM_READ_ONLY_NOTEBOOK_ID }}
68-
NOTEBOOKLM_GENERATION_NOTEBOOK_ID: ${{ secrets.NOTEBOOKLM_GENERATION_NOTEBOOK_ID }}
69-
run: |
70-
echo "READ_ONLY_NOTEBOOK_ID: $NOTEBOOKLM_READ_ONLY_NOTEBOOK_ID"
71-
echo "GENERATION_NOTEBOOK_ID: $NOTEBOOKLM_GENERATION_NOTEBOOK_ID"
72-
echo "READ_ONLY length: ${#NOTEBOOKLM_READ_ONLY_NOTEBOOK_ID}"
73-
echo "GENERATION length: ${#NOTEBOOKLM_GENERATION_NOTEBOOK_ID}"
74-
75-
- name: Debug - List artifacts in READ_ONLY notebook
76-
if: inputs.test_filter != ''
77-
env:
78-
NOTEBOOKLM_AUTH_JSON: ${{ secrets.NOTEBOOKLM_AUTH_JSON }}
79-
NOTEBOOKLM_READ_ONLY_NOTEBOOK_ID: ${{ secrets.NOTEBOOKLM_READ_ONLY_NOTEBOOK_ID }}
80-
shell: bash
81-
run: |
82-
python3 << 'EOF'
83-
import asyncio
84-
import os
85-
import traceback
86-
87-
async def main():
88-
nb_id = os.environ.get("NOTEBOOKLM_READ_ONLY_NOTEBOOK_ID")
89-
print(f"NOTEBOOKLM_READ_ONLY_NOTEBOOK_ID set: {bool(nb_id)}")
90-
print(f"NOTEBOOKLM_AUTH_JSON set: {bool(os.environ.get('NOTEBOOKLM_AUTH_JSON'))}")
91-
92-
if not nb_id:
93-
print("ERROR: NOTEBOOKLM_READ_ONLY_NOTEBOOK_ID not set")
94-
return
95-
96-
print(f"Notebook ID: {nb_id}")
97-
98-
try:
99-
from notebooklm import NotebookLMClient
100-
print("Creating client...")
101-
client = await NotebookLMClient.from_storage()
102-
print(f"Client created, auth cookies: {len(client.auth.cookies)} cookies")
103-
104-
async with client:
105-
print("Client opened, fetching artifacts...")
106-
raw = await client.artifacts._list_raw(nb_id)
107-
print(f"Total raw artifacts: {len(raw)}")
108-
109-
for i, a in enumerate(raw):
110-
if isinstance(a, list) and len(a) > 4:
111-
art_type = a[2]
112-
status = a[4]
113-
title = a[1] if len(a) > 1 else "?"
114-
length = len(a)
115-
type_names = {1: "AUDIO", 2: "REPORT", 3: "VIDEO", 4: "QUIZ/FLASH",
116-
5: "MIND_MAP", 7: "INFOGRAPHIC", 8: "SLIDE_DECK", 9: "DATA_TABLE"}
117-
type_name = type_names.get(art_type, f"TYPE_{art_type}")
118-
status_name = {1: "PROCESSING", 2: "PENDING", 3: "COMPLETED"}.get(status, f"STATUS_{status}")
119-
print(f" [{i}] {type_name} | {status_name} | len={length} | {title[:50]}")
120-
121-
if art_type == 2:
122-
has_content = len(a) > 7 and a[7] is not None
123-
print(f" → len>7: {length > 7}, status==3: {status == 3}, has_content: {has_content}")
124-
except Exception as e:
125-
print(f"ERROR: {type(e).__name__}: {e}")
126-
traceback.print_exc()
127-
128-
asyncio.run(main())
129-
EOF
130-
13164
- name: Run E2E tests
13265
env:
13366
NOTEBOOKLM_AUTH_JSON: ${{ secrets.NOTEBOOKLM_AUTH_JSON }}

src/notebooklm/data/SKILL.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,12 @@ pip install notebooklm-py
1616

1717
**From GitHub (use latest release tag, NOT main branch):**
1818
```bash
19-
# Get the latest release tag
20-
LATEST_TAG=$(gh release view --repo teng-lin/notebooklm-py --json tagName -q .tagName)
19+
# Get the latest release tag (using curl)
20+
LATEST_TAG=$(curl -s https://api.github.com/repos/teng-lin/notebooklm-py/releases/latest | grep '"tag_name"' | cut -d'"' -f4)
2121
pip install "git+https://github.com/teng-lin/notebooklm-py@${LATEST_TAG}"
2222
```
2323

24-
⚠️ **NEVER install from main branch** (`pip install git+https://github.com/teng-lin/notebooklm-py`). The main branch may contain unreleased/unstable changes. Always use PyPI or a specific release tag.
24+
⚠️ **DO NOT install from main branch** (`pip install git+https://github.com/teng-lin/notebooklm-py`). The main branch may contain unreleased/unstable changes. Always use PyPI or a specific release tag, unless you are testing unreleased features.
2525

2626
After installation, install the Claude Code skill:
2727
```bash

tests/e2e/test_downloads.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -175,13 +175,6 @@ class TestDownloadReport:
175175
@pytest.mark.readonly
176176
async def test_download_report(self, client, read_only_notebook_id):
177177
"""Downloads existing report as markdown - read-only."""
178-
# Debug: show what artifacts exist
179-
raw = await client.artifacts._list_raw(read_only_notebook_id)
180-
print(f"\n[DEBUG] Total raw artifacts: {len(raw)}")
181-
for i, a in enumerate(raw):
182-
if isinstance(a, list) and len(a) > 2:
183-
print(f" [{i}] type={a[2]}, len={len(a)}, status={a[4] if len(a) > 4 else '?'}")
184-
185178
with tempfile.TemporaryDirectory() as tmpdir:
186179
output_path = os.path.join(tmpdir, "report.md")
187180
try:

0 commit comments

Comments
 (0)