Skip to content

Preserve main branch updates for review against master - #52

Open
Coding-Dev-Tools wants to merge 27 commits into
masterfrom
main
Open

Coding-Dev-Tools wants to merge 27 commits into
masterfrom
main

Conversation

@Coding-Dev-Tools

@Coding-Dev-Tools Coding-Dev-Tools commented Oct 13, 2026 •

Copy link
Copy Markdown
Owner

Draft archival comparison for the current main branch tip 55776d1bbb0755641a1dee32e2e19c6203043f06 against the repository's default master at 272bfa8dd83b91aebeca96304404e7b64871c753. main is 9 commits ahead and 30 behind. The local Git comparison reports multiple merge bases; GitHub currently computes an 8-file PR diff. The changes need reconciliation with the newer default branch and overlap with historical merged work. Keep this draft for review; do not merge unchanged. No tests were run in this preservation pass.

Coding-Dev-Tools and others added 19 commits May 16, 2026 20:28
…aceful import fallback

- Remove revenueholdings-license>=0.1.0 from required dependencies
- Add license optional-dependencies group with revenueholdings-license
- Make require_license() import conditional: try/except ImportError
- On ImportError, define no-op require_license() and print warning
- Fixes CI breakage caused by revenueholdings-license not on PyPI (COM-79, COM-82, COM-83)
# Conflicts:
#	pyproject.toml
#	src/deadcode/cli.py
docs: add AGENTS.md
…s Revenue Holdings / stale 2026 year); W-directed fleet-wide pass
…ssors

Detect and treat `styles.card` / `styles['card-hover']` (the canonical
Next.js/React CSS-module consumption pattern) as used classes. Previously
every *.module.css class was reported as orphaned and marked removable=True,
which could delete live styles. Bracket accessors are recognized and
non-module object access (util.foo) is not over-matched.

Adds TestCSSModuleUsage regression tests; all 116 tests pass, ruff clean.

Co-Authored-By: cowork-bot <cowork-bot@users.noreply.github.com>
The previous SHA 34e1148 was a v6-era PR commit, not the v4 tag.
GitHub Actions could not resolve it, causing zero-step CI failures.
The correct v4 tag SHA is 11d5960a326750d5838078e36cf38b85af677262.
Validated local candidate: 57daaacaffc6dfed6cfc4458edc6e2e889c138c1

- 57daaacaffc6dfed6cfc4458edc6e2e889c138c1 fix(scanner): preserve module styles and enforce project boundary
Validated local candidate: 33b48f9efde1ecd6207aea0fca07cd6b4560bb2e

- 33b48f9efde1ecd6207aea0fca07cd6b4560bb2e fix(cli): validate remove target and organize imports
@github-actions

Copy link
Copy Markdown

Automated Code Review

Ruff lint: 0 issue(s)

Ruff format: Clean

Secret detection: 0 potential secret(s)

Verdict: PASS

Automated by Coding-Dev-Tools/.github reusable workflow.

@Coding-Dev-Tools
Coding-Dev-Tools marked this pull request as ready for review October 17, 2026 13:00
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 17, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-17T14:16:49.539463Z 9dbded5 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

This branch had an error being deployed

1 failed (outdated) deployment
github-pages — 08a9f3c9 Deployed Oct 17, 2026 by Coding-Dev-Tools via deploy #47
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.

3 participants