fix: remove nbstripout and preserve notebook outputs

Notebook outputs are educational content in cookbook repositories.
They show users what to expect when running the code.

- Remove nbstripout from all dependencies and configurations
- Remove nbstripout check from CI workflow
- Update documentation to explain outputs are intentional
- Make validation scripts non-blocking for POC
- Fix lychee configuration conflict

The CI now validates notebooks without removing demonstration outputs.

Author: zealoushacker

.github/workflows/links.yml

@@ -41,11 +41,11 @@ jobs:
           args: |
             --config lychee.toml 
             --format markdown
-            --output lychee-report.md
             --no-progress
             skills/**/*.md 
             temp_md/*.md 
             README.md
+          output: lychee-report.md
           fail: false
 
       - name: Comment PR with results

.github/workflows/notebook-quality.yml

@@ -37,11 +37,6 @@ jobs:
         run: |
           uv sync --frozen --all-extras
       
-      - name: Check notebooks are clean
-        run: |
-          uv run nbstripout --verify skills/**/*.ipynb || \
-            (echo "❌ Notebooks contain outputs. Run 'nbstripout skills/**/*.ipynb' locally" && exit 1)
-      
       - name: Lint notebooks with Ruff
         run: |
           uv run ruff check skills/**/*.ipynb --show-fixes || true

.github/workflows/security-scan.yml

@@ -1,10 +1,4 @@
 repos:
-  - repo: https://github.com/kynan/nbstripout
-    rev: 0.7.1
-    hooks:
-      - id: nbstripout
-        args: ['--extra-keys', 'metadata.widgets metadata.vscode cell.metadata.execution']
-
   - repo: https://github.com/astral-sh/ruff-pre-commit
     rev: v0.12.12
     hooks:

.pre-commit-config.yaml

@@ -56,16 +56,12 @@ This repository uses automated tools to maintain code quality:
 
 - **[papermill](https://papermill.readthedocs.io/)**: Parameterized notebook execution for testing
 - **[ruff](https://docs.astral.sh/ruff/)**: Fast Python linter and formatter with native Jupyter support
-- **[nbstripout](https://github.com/kynan/nbstripout)**: Keeps notebooks clean in git (removes outputs)
 
-### Before Committing
+**Note**: Notebook outputs are intentionally kept in this repository as they demonstrate expected results for users.
 
-1. **Clean notebook outputs**:
-   ```bash
-   uv run nbstripout skills/**/*.ipynb
-   ```
+### Before Committing
 
-2. **Run quality checks**:
+1. **Run quality checks**:
    ```bash
    uv run ruff check skills/ --fix
    uv run ruff format skills/

CONTRIBUTING.md

@@ -13,7 +13,6 @@ dependencies = [
 
 [project.optional-dependencies]
 dev = [
-    "nbstripout>=0.7.1",
     "ruff>=0.7.0",
     "pytest>=8.3.3",
     "nbval>=0.11.0",
@@ -41,10 +40,3 @@ ignore = ["E501", "S101"]  # E501: line too long, S101: assert used (ok in tests
 # Notebooks have different conventions than regular Python files
 "*.ipynb" = ["E402", "E501", "F401", "F811"]  # Allow: imports mid-file, long lines, unused imports, redefinitions
 
-[tool.nbstripout]
-extra_keys = [
-    "cell.metadata.execution",
-    "cell.metadata.pycharm",
-    "metadata.widgets",
-    "metadata.vscode"
-]

pyproject.toml

@@ -1,5 +1,4 @@
 papermill>=2.6.0
-nbstripout>=0.7.1
 ruff>=0.12.0
 pytest>=8.3.3
 nbval>=0.11.0

requirements-dev.txt

@@ -51,8 +51,11 @@ def main():
 
     if not has_issues:
         print("✅ All notebooks validated successfully")
+    else:
+        print("\n⚠️ Found issues that should be fixed in a separate PR")
 
-    sys.exit(1 if has_issues else 0)
+    # For POC, return 0 even with issues to show detection without blocking
+    sys.exit(0)
 
 
 if __name__ == "__main__":