Skip to content

Campaign 253/migrate gitlab bridge deno - #64

Merged
hyperpolymath merged 14 commits into
mainfrom
campaign-253/migrate-gitlab-bridge-deno
Jul 25, 2026
Merged

Campaign 253/migrate gitlab bridge deno#64
hyperpolymath merged 14 commits into
mainfrom
campaign-253/migrate-gitlab-bridge-deno

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Summary

Closes #

Type of change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (would change existing behaviour)
  • 🕳️ Soundness fix (fixes a checker/proof false-negative)
  • 📖 Documentation
  • 🧹 Refactor / tech debt (behaviour-preserving)
  • ⚡ Performance
  • 🔧 Build / CI / tooling

How has this been verified?

Checklist

  • My commits are signed (git commit -S).
  • I ran the project's own checks/tests locally and they pass.
  • New files carry the correct SPDX-License-Identifier (code/config MPL-2.0,
    prose CC-BY-SA-4.0); I did not relicense existing files.
  • Docs are updated, and no public claim now overstates what the code does.
  • I have not introduced a soundness hole (or I have flagged where I might have).

Notes for reviewers

hyperpolymath and others added 14 commits May 30, 2026 21:55
Class B (npm-wrapper via Deno) for the gitlab-bridge/ sub-tree.

- Deletes redundant package.json (self-documented as a "minimal manifest
  for the ReScript compiler toolchain only").
- Updates deno.json tasks build|clean to use `deno run -A
  --node-modules-dir=auto npm:rescript` so rescript's compiler binary
  can still resolve @rescript/core via node_modules without needing a
  committed package.json stub.

Refs: standards#253 STEP 3 (smallest-first batch).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- C001: CodeQL language fixes
- C002: License identifier standardization
- C003: Outdated actions audit
- C004: Pin standards refs to SHA 861b5e9
- C005: Add workflow-level permissions
- Remove execute_js tool from MV2 and MV3 background scripts to eliminate
  eval code execution vulnerability (S2083)
- Remove EXECUTE_IN_PAGE handler from content bridge script to eliminate
  eval code execution vulnerability (S2083)
- Replace innerHTML with DOM manipulation methods in gecko-browser-extension
  content.js, popup.js, and mozilla-extension sidebar.js (S2639)

These changes address Security Rating on New Code < A in PR #64.

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
Resolves merge conflicts between PR branch and main.

Generated by Mistral Vibe.
Co-Authored-By: Mistral Vibe <vibe@mistral.ai>
@hyperpolymath
hyperpolymath force-pushed the campaign-253/migrate-gitlab-bridge-deno branch from 964524d to 73d2e5c Compare July 25, 2026 22:15
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
C Security Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@hyperpolymath
hyperpolymath merged commit 4098983 into main Jul 25, 2026
1 of 4 checks passed
@hyperpolymath
hyperpolymath deleted the campaign-253/migrate-gitlab-bridge-deno branch July 25, 2026 22:17
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