Skip to content

Mark vendored and byte-gated trees; document the ops/ taxonomy (#78 phase 0) - #80

Draft
qywu wants to merge 1 commit into
mainfrom
reorg/phase0-vendored-markers
Draft

Mark vendored and byte-gated trees; document the ops/ taxonomy (#78 phase 0)#80
qywu wants to merge 1 commit into
mainfrom
reorg/phase0-vendored-markers

Conversation

@qywu

@qywu qywu commented Aug 21, 2026

Copy link
Copy Markdown
Member

Phase 0 of #78 — mark, don't move. Zero import churn; no Python file is modified.

  • Hook protection: the pre-commit global exclude (already covering quack/) now also covers linear_attention/flashqla/ and bi_families_v2.py. The latter was only excluded from ruff-format before — but the ruff hook runs --fix-only (a rewriter), so a rule upgrade could have silently edited the sha256-gated bytes.
  • Direct-invocation protection: the same excludes land in [tool.ruff] with force-exclude = true, so ruff skips these paths even when they're passed explicitly (this is exactly how a tree-wide ruff --fix recently rewrote 45 vendored quack files).
  • codespell skips the vendored trees.
  • ops/quack/VENDORED.md: provenance (Dao-AILab QuACK, snapshotted at 0.4.1 as a partial fork that still imports helpers from the quack-kernels==0.5.0 PyPI pin) and the local-patch ledger (cute_dsl_elf_fix, cute_dsl_mlir_threading), generalizing the existing flashqla pattern.
  • ops/README.md: the edit-policy table (vendored / byte-contract-gated / first-party), a map of every subtree, and where each is headed under Reorganize src/xorl/ops: quarantine vendored code, separate kernels / layers / objectives, one home for the exact-contract family #78.

Verified: pre-commit run --all-files (the lint CI command) passes; ruff check on the excluded paths now reports no files.

Next: phase 1 (physically quarantine quack/ + flashqla/ under ops/_vendored/ with re-export stubs) after #77 lands.

…hase 0)

No code moves — this is the mark-don't-move phase of #78:

- Extend the pre-commit global exclude (which already covered quack/) to
  flashqla/ and bi_families_v2.py, so no rewriting hook can touch them.
  bi_families_v2.py was previously excluded only from ruff-format; the ruff
  hook runs --fix-only and could have silently rewritten the sha256-gated
  bytes on a rule upgrade.
- Add the same excludes to [tool.ruff] in pyproject with force-exclude, so
  direct ruff invocations (which receive explicit file paths and ignore
  ordinary excludes) skip them too.
- Skip the vendored trees in codespell.
- Add src/xorl/ops/quack/VENDORED.md (provenance: Dao-AILab QuACK 0.4.1
  partial fork over the quack-kernels PyPI pin; local-patch ledger:
  cute_dsl_elf_fix, cute_dsl_mlir_threading) generalizing the existing
  flashqla VENDORED.md pattern.
- Add src/xorl/ops/README.md: the edit-policy table (vendored /
  byte-contract-gated / first-party), the current map of every subtree, and
  where each is headed under #78.
@broly-code-security-scanner

Copy link
Copy Markdown

Broly Security Scan

Note

Clean scan
No vulnerabilities detected in this PR.

Note

Re-scan this PR anytime with /broly scan — useful after /broly undismiss, or to refresh findings without a new push.

Broly — SAST (zai-org/GLM-5.2) · Secrets · SCA · IaC · GH Actions · Base Images · Supply Chain Threats · Exploit Chains · Adversarial Verification

We're continuously improving Broly's accuracy and finding quality — your feedback is valuable. False positives, missed findings, bugs, and feature requests all welcome.

Ask in #security-engineering   Powered by Together AI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant