Skip to content

chore: fill or remove RSR template placeholders - #68

Merged
hyperpolymath merged 3 commits into
mainfrom
fix/rsr-placeholders
Jul 28, 2026
Merged

chore: fill or remove RSR template placeholders#68
hyperpolymath merged 3 commits into
mainfrom
fix/rsr-placeholders

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

openssf-compliance.yml fails when any of the thirteen files it checks still contains a {{PLACEHOLDER}} token. This clears them, with no invention.

  • Deleted the TEMPLATE INSTRUCTIONS (delete this block before publishing) comment — the template says to delete it, and it is where every legend line lived, so a large share of the reported tokens were the file documenting its own placeholders.
  • Filled what the repository itself supplies: owner and repo from the git remote, project name, year, forge, main branch, contact email.
  • Removed PGP and website lines. https://github.com/<user>.gpg returns HTTP 200 for every account; with no key uploaded the body is a stub reading "This user hasn't uploaded any GPG keys". No key is published for either account, and commit signing here is SSH — unrelated. The template sanctions this: "Optional: Remove sections that don't apply (e.g. PGP if you don't use it)."\n\nA security policy telling a researcher to encrypt to a key that does not exist is worse than one that does not mention encryption.\n\nWhere applicable, this also fixes a misrouted advisory URL. A previous just init copied already-filled templates out of squisher-corpus, so 51 repositories directed vulnerability reports to hyperpolymath/squisher-corpus — a repository unrelated to the code being reported, meaning the affected maintainer never saw them. Repointed from this repository's own git remote.

hyperpolymath and others added 3 commits July 26, 2026 14:53
openssf-compliance.yml fails when any of the thirteen files it checks
still contains a {{PLACEHOLDER}} token. This clears them.

Three kinds of change, no invention:

The "TEMPLATE INSTRUCTIONS (delete this block before publishing)" comment
is deleted. The template says to delete it, and it is where every legend
line lives -- so a large share of the reported tokens were the file
documenting its own placeholders, not real unfilled fields.

Tokens derivable from the repository are filled: owner and repo from the
git remote, project name, year, forge, main branch, contact email.

PGP and website lines are removed rather than filled, because nothing
true could go in them. https://github.com/<user>.gpg returns HTTP 200 for
every account; with no key uploaded the body is a stub reading "This user
hasnt uploaded any GPG keys". No key is published for either account
here, and commit signing in this estate is SSH, which is unrelated. Only
one repository in the estate has a domain, so {{WEBSITE}} likewise has no
correct value. The template sanctions this: "Optional: Remove sections
that dont apply (e.g. PGP if you dont use it)." A security policy telling
a researcher to encrypt to a key that does not exist is worse than one
that does not mention encryption.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
@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

Comment thread CONTRIBUTING.md
Comment thread guix.scm
Comment thread guix.scm
Comment thread CODE_OF_CONDUCT.md
Comment thread SECURITY.md
@gitar-bot

gitar-bot Bot commented Jul 28, 2026

Copy link
Copy Markdown

Note

Automatic reviews are paused because your trial's included automatic processing has been used for this period. Upgrade now, or comment "Gitar review" to run a review anytime.
Learn more

CI failed: 4 CI job failures caused by configuration, tooling, and governance issues including an unresolved cache path, a banned ReScript file, missing pyproject.toml for Poetry, and an unpinned GitHub action.

Overview

Multiple distinct configuration, tooling, and governance checks failed across 4 CI jobs. All failures are directly related to the repository structure or workflow configuration.

Failures

GitHub Actions Cache Path Unresolved (confidence: high)

  • Type: configuration
  • Affected jobs: 90327336176
  • Related to change: yes
  • Root cause: The caching action received a path parameter that does not exist in the repository or workspace at runtime.
  • Suggested fix: Verify and correct the specified cache paths in the workflow configuration to ensure all target files or directories exist.

Hypatia Governance Banned Language File (confidence: high)

  • Type: tooling
  • Affected jobs: 90327338965
  • Related to change: yes
  • Root cause: The file frontend/src/graphql/Queries.res was detected, but ReScript files (.res) are banned by governance rules requiring AffineScript instead.
  • Suggested fix: Remove or rewrite the ReScript file using AffineScript, or add a proper exemption pragma.

Poetry Missing pyproject.toml (confidence: high)

  • Type: configuration
  • Affected jobs: 90327336333
  • Related to change: yes
  • Root cause: Poetry could not locate a pyproject.toml file in the backend directory or its parents due to missing files or incorrect working directory settings.
  • Suggested fix: Ensure pyproject.toml is present in the expected backend directory or adjust the working directory path in the workflow file.

Unpinned GitHub Action (confidence: high)

  • Type: tooling
  • Affected jobs: 90327339050
  • Related to change: yes
  • Root cause: .github/workflows/ci.yml uses an unpinned action (aquasecurity/trivy-action@master), violating the repository policy requiring actions to be pinned by full commit SHA.
  • Suggested fix: Replace aquasecurity/trivy-action@master with its full commit SHA equivalent in .github/workflows/ci.yml.

Summary

  • Change-related failures: 4 configuration and tooling failures (cache path issue, banned ReScript file, missing pyproject.toml, and unpinned GitHub action)
  • Infrastructure/flaky failures: 0
  • Recommended action: Fix the workflow file configurations (cache paths, action pinning), provide the required pyproject.toml or adjust backend paths, and remove or exempt the banned ReScript file.
Code Review ✅ Approved 5 resolved / 5 findings

Removes template placeholders across documentation files to satisfy openssf-compliance.yml, but leaves CONTRIBUTING.md truncated, CODE_OF_CONDUCT.md with unremoved instructions, and introduces metadata mismatches in guix.scm.

✅ 5 resolved
Bug: guix.scm describes squisher-corpus in social-media-polygraph repo

📄 guix.scm:2 📄 guix.scm:10 📄 guix.scm:14-16
This repo is social-media-polygraph, but guix.scm now names the package squisher-corpus with matching synopsis, description and home-page https://github.com/hyperpolymath/squisher-corpus. This appears to be the wrong template copied in (the same squisher-corpus misrouting the PR aims to fix elsewhere). Set name/synopsis/description/home-page back to social-media-polygraph.

Quality: CONTRIBUTING.md is truncated and malformed

📄 CONTRIBUTING.md:1-15 📄 CONTRIBUTING.md:111-116
The new CONTRIBUTING.md is corrupted: it opens with an H1 # Clone the repository followed by raw shell commands (no opening bash fence), has an orphan closing at line 16, and the file ends abruptly at line 116 inside an unterminated code block. The intro (real title/prerequisites and opening fence) and the closing content are missing, so the document renders incorrectly. Restore the full template body with matched code fences and a proper document title/heading structure.

Bug: guix.scm license constructor likely called with wrong arity

📄 guix.scm:17-18
((@@ (guix licenses) license) "PMPL-1.0-or-later" "https://...") reaches into the private license record constructor, which takes three fields (name, uri, comment); passing only two args will raise a wrong-number-of-arguments error and break guix shell -f guix.scm. Use the public API — either an existing license binding or (license "PMPL-1.0-or-later" "https://github.com/hyperpolymath/palimpsest-license" "Palimpsest license") with all three fields — after importing it properly rather than via @@.

Quality: CODE_OF_CONDUCT.md still contains TEMPLATE INSTRUCTIONS block

📄 CODE_OF_CONDUCT.md:3-17
The PR description states this block was deleted, but the published file still begins with the TEMPLATE INSTRUCTIONS (delete this block before publishing) comment, which explicitly instructs its own removal. Delete lines 3-21 so the document does not ship template scaffolding.

Quality: SECURITY.md retains PGP section with unfilled 'TBD' fingerprint

📄 SECURITY.md:41-55 📄 SECURITY.md:355
The PR summary says PGP/website lines were removed because no key exists, yet SECURITY.md still advertises a PGP key download, a curl | gpg --import flow, and Fingerprint: TBD. This directs reporters to encrypt to a nonexistent key and leaves an unfilled TBD placeholder. Either remove the PGP section as the description claims, or fill in a real fingerprint/key.

Tip

Comment Gitar fix CI to trigger a fix.

Options

Display: compact → Showing less information.

Comment with these commands to change the behavior for this request:

Compact
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

@hyperpolymath
hyperpolymath merged commit 5b782ae into main Jul 28, 2026
20 of 25 checks passed
@hyperpolymath
hyperpolymath deleted the fix/rsr-placeholders branch July 28, 2026 16:18
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