Copilot/sync rabbit data

[5hy7xz92nd-oss]

No description provided.

[Copilot]

Pull request overview

Copilot reviewed 3 out of 17 changed files in this pull request and generated no new comments.

Suppressed comments (4)

validate_pocs.py:11

• Tuple is imported but never used; this can trigger lint/type-check failures and makes the module look unfinished. Remove the unused import (or use it if intended).

from typing import List, Dict, Tuple

tests/test_validate_pocs.py:18

• This test is brittle: asserting len(python_targets) > 20 will fail as the repo grows/shrinks even when discovery is correct. Prefer only asserting non-empty discovery and/or presence of a stable known target.

        self.assertTrue(any(path.name == "run_demo.py" for path in python_targets))
        self.assertTrue(any(path.name == "package.json" for path in package_targets))
        self.assertGreater(len(python_targets), 20)
        self.assertGreater(len(package_targets), 0)

.github/instructions/*.instructions.md:5

• This new Copilot instructions file appears to contain large amounts of non-instructional/log-like content (mentions, repeated ASCII art, and external chat links). Keeping this in .github/instructions risks prompt-injection style behavior and makes repository instructions hard to review and maintain. Consider removing it, or replacing it with a short, task-focused set of instructions under a normal filename (avoid * in filenames, which breaks checkouts on Windows/filesystems that disallow it).

@5hy7xz92nd-oss @copilot @copilot @copilot @we-tech-company @copilot @5hy7xz92nd-oss 🕴️🔃〰️⤴️@copilot🌎 REALITY
|
↓
👁️ Observer
|

validate_pocs.py:14

• discover_*_targets() only excludes .git and __pycache__, so it will also traverse common local/CI artifact directories like node_modules/ and virtualenvs (several PoC folders already ignore .venv/venv and node_modules in their .gitignore). This can significantly slow validation and may surface irrelevant errors from installed dependencies. Expand EXCLUDED_DIRS to skip those artifact directories.

EXCLUDED_DIRS = {".git", "__pycache__"}

[Copilot]

Pull request overview

Copilot reviewed 3 out of 17 changed files in this pull request and generated no new comments.

Suppressed comments (2)

validate_pocs.py:11

• Tuple is imported but never used, which can trigger linter failures and adds noise to the module imports.

from typing import List, Dict, Tuple

tests/test_validate_pocs.py:18

• These tests are coupled to the current repository contents (expects specific filenames and minimum counts, and asserts the whole repo validates cleanly). This will become flaky as PoCs are added/removed or if the repo intentionally contains non-compiling examples; prefer testing against a temporary directory fixture with controlled files.

        self.assertTrue(any(path.name == "run_demo.py" for path in python_targets))
        self.assertTrue(any(path.name == "package.json" for path in package_targets))
        self.assertGreater(len(python_targets), 20)
        self.assertGreater(len(package_targets), 0)