Copilot/auto sync updates

[5hy7xz92nd-oss]

No description provided.

[Copilot]

Pull request overview

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

Suppressed comments (3)

tests/test_validate_pocs.py:11

• The unittest suite implicitly requires node to be installed (because validate_repo validates JS via node --check), which will cause the entire test run to fail in Python-only environments. Consider skipping these tests when node is unavailable (CI already installs it).

    def setUp(self) -> None:
        self.repo_root = Path(__file__).resolve().parents[1]

.github/instructions/*.instructions.md:3

• This file name contains a literal * character (.github/instructions/*.instructions.md), which breaks checkouts on Windows (invalid filename) and is likely accidental. The file contents also appear to be unrelated PR/chat logs and emoji spam rather than actionable Copilot instructions—please remove it or replace it with concise repository instructions under a valid filename.

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

validate_pocs.py:284

• Typo in the discovery sanity error message: “validateable” should be “validatable” (this string is user-facing and will be printed on failure).

            "discovery returned no validateable files despite PoC content on disk; "

[Copilot]

Pull request overview

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

Suppressed comments (4)

tests/test_validate_pocs.py:11

• Most tests depend on Node being available (because fixtures include .js files and validate_repo() enforces node --check). Add a setUp() guard to skip these tests when node is not installed, so the suite fails for real validation issues rather than missing local tooling.

    def setUp(self) -> None:
        self.repo_root = Path(__file__).resolve().parents[1]

tests/test_validate_pocs.py:4

• Tests call validate_repo() which requires node when any .js fixtures are present, but the test module never imports shutil (needed to detect node) and currently can't skip cleanly when Node isn't installed.

import json
import tempfile
import unittest
from pathlib import Path

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

• This Copilot instructions file appears to contain large amounts of unrelated/duplicated text (including external links) and is named literally *.instructions.md (with a * in the filename). This is likely accidental or autogenerated noise and can interfere with tooling that consumes .github/instructions content; please remove it or replace it with concise, repo-relevant instructions.

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

validate_pocs.py:101

• py_compile.compile(..., doraise=True) can raise OSError (e.g., unreadable/missing file). Right now that would crash the validator instead of reporting a collected error like the other validators do.

        try:
            py_compile.compile(str(path), doraise=True)
        except py_compile.PyCompileError as exc:  # pragma: no cover - covered via broken fixtures
            errors.append(f"{path.relative_to(root)}: {exc}")