Skip to content

v1.1.0#59

Open
taarikashenafi wants to merge 2 commits into
mainfrom
taarikashenafi-release-plugin-v1-1-0
Open

v1.1.0#59
taarikashenafi wants to merge 2 commits into
mainfrom
taarikashenafi-release-plugin-v1-1-0

Conversation

@taarikashenafi

@taarikashenafi taarikashenafi commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Included

Excluded

  • Hosted workflow migration in .github/workflows/scan-static-sites.yml
  • Scanner npm-loading changes (tracked independently for scanner v3.4.0)

Validation

  • Build, typecheck, lint, and format
  • 163 tests on Node 26; pre-conflict release candidate also passed its full suite on Node 22 and 24, with the refreshed CI matrix covering all supported versions
  • Credential-free defaults and fake-judge opt-in path
  • URL/HTML redaction, keyword-stuffing cases, and generated-remediation cases
  • Configuration schema assertions
  • npm pack contents and isolated tarball import

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 420c7633-4b5c-4b35-a1bd-5cb6e438a8e4

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Prepares the package and documentation for the v1.1.0 release.

Changes:

  • Bumps package metadata to 1.1.0.
  • Pins the scanner example to 1.1.0.
  • Distinguishes default deterministic rules from opt-in model-backed checks.
Show a summary per file
File Description
README.md Updates release usage and rule documentation.
package.json Bumps package version.
package-lock.json Synchronizes lockfile version metadata.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 2/3 changed files
  • Comments generated: 1
  • Review effort level: Medium

Comment thread README.md
- 🚧 Catch boilerplate placeholder `alt` text (`"todo"`, `"tbd"`, `"fixme"`)
- ♿ Catch images missing an `alt` attribute entirely (without flagging intentional decorative `alt=""`)

An additional opt-in, model-backed `alt-text-quality` rule evaluates whether alt text accurately describes its image and flags SEO keyword stuffing. It is disabled by default and requires a GitHub Models token.
@taarikashenafi taarikashenafi changed the title Prepare v1.1.0 release v1.1.0 Jul 24, 2026
@taarikashenafi
taarikashenafi requested a review from JoyceZhu July 24, 2026 19:07
JoyceZhu
JoyceZhu previously approved these changes Jul 24, 2026

@JoyceZhu JoyceZhu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the README might need some merge conflict resolution (Keenan's PR updating the docs just automerged and also rearranged that file a decent amount), but LGTM otherwise!

Resolve the README conflict after the remediation and documentation changes merged to main.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 420c7633-4b5c-4b35-a1bd-5cb6e438a8e4
Copilot AI review requested due to automatic review settings July 24, 2026 21:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review details

  • Files reviewed: 2/3 changed files
  • Comments generated: 0 new
  • Review effort level: Medium

@taarikashenafi
taarikashenafi requested a review from JoyceZhu July 24, 2026 21:35
@taarikashenafi

Copy link
Copy Markdown
Contributor Author

I think the README might need some merge conflict resolution (Keenan's PR updating the docs just automerged and also rearranged that file a decent amount), but LGTM otherwise!

fixed the merge conflict and added Keenan's latest changes

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