Skip to content

docs(office): drop stale DIST-STATUS banner (devin-office published) - #101

Merged
Icaro0310 merged 2 commits into
mainfrom
fix/office-dist-status-stale
Nov 9, 2026
Merged

Icaro0310 merged 2 commits into
mainfrom
fix/office-dist-status-stale

Conversation

@Icaro0310

@Icaro0310 Icaro0310 commented Nov 9, 2026 •

Copy link
Copy Markdown
Owner

What

Removes the leftover <!-- DIST-STATUS:BEGIN ... END --> source-only banner from packages/office/README.md.

Why

devin-office is published to PyPI (0.1.0) and the devin-powerups registry-drift check (tools/check_dist_status.py) requires distribution_status: published entries to carry no DIST-STATUS markers. The banner predates the PyPI publish / monorepo absorption and currently fails the check:

error: devin-office: published but README still carries a DIST-STATUS marker
dist-status check: 1 problem(s)

Verification

python3 tools/check_dist_status.py --root <ecosystem-root> → dist-status check: 0 problem(s) (exit 0).

Generated with Devin & QA Engineer Icaro Galvao


Devin Review

…published

devin-office is published to PyPI (0.1.0) and the devin-powerups
registry-drift check requires published entries to carry no
DIST-STATUS markers. The leftover source-only banner was written
before the monorepo absorption and now fails the check.

Generated with [Devin](https://devin.ai) & QA Engineer Icaro Galvao

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@github-actions github-actions Bot added the documentation Improvements or additions to documentation label Nov 9, 2026
@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown

Static scan (advisory) — result: EXECUTED

Deterministic scan of 1 changed file(s) — gitleaks 8.30.1 · semgrep 1.182.1 · ruff 0.17.3. Findings are advisory evidence for the judge, not a lane verdict.

gitleaks — 0 finding(s)

semgrep — 0 finding(s)

ruff — skipped (no Python files changed)

@devin-ai-integration devin-ai-integration Bot 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.

Devin Review found 1 potential issue.

Devin Review

Comment thread packages/office/README.md
Comment on lines -73 to -80
<!-- DIST-STATUS:BEGIN — generated from devin-powerups/registry.json -->
> **Source-only distribution.** This project is not published to a package
> registry. Run it from a checkout:
>
> ```bash
> git clone https://github.com/Icaro0310/devin-control.git
> ```
<!-- DIST-STATUS:END -->

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.

🟡 Changelog do Office mantém instruções de distribuição obsoletas

Após a remoção do aviso, o changelog ainda afirma que o README o exibe e que não existe publicação em registro. Isso contradiz o estado publicado e descumpre a exigência de atualizar o changelog para o PR.

Learn more

O changelog do Office registra, na seção Unreleased, que o README apresenta um aviso de distribuição apenas pelo código-fonte e que não existe publicação em registro. Este PR remove esse aviso por estar obsoleto, mas não atualiza o changelog. A regra de contribuição exige atualizar o CHANGELOG antes de abrir um PR; manter a entrada antiga também documenta o estado oposto ao atual.

Example: Um leitor consulta a seção Unreleased após a mudança e encontra a afirmação de que o README contém o banner, embora o banner tenha sido removido e o pacote esteja publicado.

Recommended fix: Atualize packages/office/CHANGELOG.md na seção Unreleased para registrar a remoção do aviso obsoleto e evitar afirmar que a distribuição continua restrita ao código-fonte.

Devin Review


Was this helpful? React with 👍 or 👎 to provide feedback.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Fixed in 4a5b382 - replaced the stale Unreleased line that still claimed the README carries the DIST-STATUS banner and that there is no registry publication. It now records the banner removal and the PyPI (0.1.0) published state.

@Icaro0310

Copy link
Copy Markdown
Owner Author

devin-judge (ACP, high model): approve — {"samples": 1, "blocks": 0, "probs": [{"secrets": 0.01, "unsafe_code": 0.0, "deception": 0.05}]}

The Unreleased section still claimed the README shows the source-only
DIST-STATUS banner and no registry publication — both stale now that
devin-office is on PyPI. Flagged by Devin Review on #101.

Generated with [Devin](https://devin.ai) & QA Engineer Icaro Galvao

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown

Static scan (advisory) — result: EXECUTED

Deterministic scan of 2 changed file(s) — gitleaks 8.30.1 · semgrep 1.182.1 · ruff 0.17.3. Findings are advisory evidence for the judge, not a lane verdict.

gitleaks — 0 finding(s)

semgrep — 0 finding(s)

ruff — skipped (no Python files changed)

@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown

Merge automático bloqueado: o Judge não aprovou esta alteração. Requer revisão humana.

devin-judge/review on 4a5b382: missing (approve requires success). Quando o Judge aprovar, este gate re-corre sozinho e libera a aprovação. Um humano pode sobrepor na UI do GitHub assumindo a responsabilidade.
judge-blocked head:4a5b382b1f001b5a129fc8dfc9b1abda2eda757b

@Icaro0310

Copy link
Copy Markdown
Owner Author

devin-judge (ACP, high model): approve — {"samples": 1, "blocks": 0, "probs": [{"secrets": 0.02, "unsafe_code": 0.02, "deception": 0.05}]}

@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown

gate-resolved-via: fallback — 1 blocking thread(s) cleared by replies citing verified fix commits, with the Devin Review status on 4a5b382 re-verified clean:

  • thread PRRT_kwDOUzxHgc61OdAt via commit 4a5b382

Threads left unresolved — the findings stay visible.
gate-fallback-pairs: PRRT_kwDOUzxHgc61OdAt|commit:4a5b382 head:4a5b382b1f001b5a129fc8dfc9b1abda2eda757b

@Icaro0310
Icaro0310 merged commit e152823 into main Nov 9, 2026
30 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant