Repository navigation
docs: post-PR-48/#51 council audit fixes (T1+T2) - #53
Merged
Merged
Conversation
Fixes flagged by the 3-agent doc-review council after PR #48 (i2v/r2v + model picker) and PR #51 (GFLOW_CLI_LOCALE env override) landed on develop. Tier 1 — fictional/stale claims removed: - llms.txt: "video I2V and batch queued for Phase B" was false post-PR #48. Updated to reflect T2V/I2V/R2V shipped; only `video batch` (manifest runner) is still queued, with shell for-loop workaround pointed to. - PLAN.md: "Issue #24 completed in v0.8.1" was false — PR #51 landed 2026-05-24, post-v0.8.1, on develop only. Updated to state Phase 1 (env override) shipped + live-verified in pt-BR; selector-invariant capture still pending. Same correction to Phase 7 checklist. - KNOWN_ISSUES.md (#24): added Progress block noting PR #51 + PR #48 changes and the path to dropping `--lang=en-US`. - LIVE_VERIFICATION_video_download.md: dropped stale "I2V/R2V raise NotImplementedError" — they ship in PR #48. - docs/USAGE.md `gflow video batch`: was documenting a full TSV manifest as if working; replaced with a not-yet-implemented warning and a shell for-loop workaround (separate project_ids per shot). Tier 2 — important polish: - GFLOW_CLI_LOCALE documented in `.env.template` and `docs/CONFIGURATION.md` (it was code-only after PR #51). - CHANGELOG `[Unreleased] § Added` entry for GFLOW_CLI_LOCALE. - README + PROJECT_STATUS opening lines reframed: v0.8.1 is the latest PyPI release but develop has moved on — added a "develop (unreleased, post-v0.8.1)" pointer covering PR #37 / #46 / #48 / #51. - KNOWN_ISSUES Resolved entry for the silent `image --model` no-op fixed in PR #48 — gives users who hit it on v0.7.0-v0.8.1 a breadcrumb. All gates pass: doc-link checker (9 files), repo hygiene (255 files), ruff. Council reports kept locally at tmp/council/0{1,2,3}-*.md (gitignored). Signed-off-by: Flavio Oliva <ffroliva@gmail.com>
15990166686ll-crypto
pushed a commit
to 15990166686ll-crypto/gflow-cli
that referenced
this pull request
Aug 24, 2026
docs: post-PR-48/ffroliva#51 council audit fixes (T1+T2)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Doc-drift fixes flagged by the 3-agent
/gflow:doc-reviewcouncil after PR #48 (i2v/r2v + model picker) and PR #51 (GFLOW_CLI_LOCALE env override) landed ondevelop. No code touched — only.md,.txt,.template.Tier 1 — fictional / stale claims (4 fixed + 1 retracted doc claim)
llms.txt:3— "video I2V and batch queued for Phase B" was false post-PR feat(video): wire i2v/r2v + model/duration/count picker; fix image --model no-op #48. Updated; onlygflow video batch(manifest runner) is still queued.PLAN.md:342+:348— "Issue Make UiAutomationTransport selectors language-agnostic (structural-first) #24 completed in v0.8.1" was false. PR feat: locale-overridable Playwright launch (issue #24 prep) #51 landed today, post-v0.8.1, on develop only. Updated to state Phase 1 (env override) shipped + live-verified in pt-BR; locale-invariant selector capture still pending.KNOWN_ISSUES.md(Make UiAutomationTransport selectors language-agnostic (structural-first) #24) — added Progress block citing PR feat: locale-overridable Playwright launch (issue #24 prep) #51 + PR feat(video): wire i2v/r2v + model/duration/count picker; fix image --model no-op #48 and the path to dropping--lang=en-US.docs/LIVE_VERIFICATION_video_download.md— dropped stale "I2V/R2V raise NotImplementedError" (they shipped in PR feat(video): wire i2v/r2v + model/duration/count picker; fix image --model no-op #48).docs/USAGE.md § gflow video batch— was documenting a full TSV manifest as if working. Replaced with a not-yet-implemented warning + a shell for-loop workaround (separateproject_idper shot, but videos still get generated).Tier 2 — important polish
GFLOW_CLI_LOCALEdocumented in.env.templateanddocs/CONFIGURATION.md(was code-only after PR feat: locale-overridable Playwright launch (issue #24 prep) #51).CHANGELOG.md [Unreleased] § Addedentry forGFLOW_CLI_LOCALE.develophas moved on (PR refactor: fix SonarCloud issues — function length, exception types, unused params #37 / docs(readme): restore Stats section + extra badges (for v0.9.0) #46 / feat(video): wire i2v/r2v + model/duration/count picker; fix image --model no-op #48 / feat: locale-overridable Playwright launch (issue #24 prep) #51).KNOWN_ISSUES.md § Resolved— added entry for the silentgflow image t2i --modelno-op fixed in PR feat(video): wire i2v/r2v + model/duration/count picker; fix image --model no-op #48 (gives users who hit it on v0.7.0–v0.8.1 a breadcrumb).Evidence
GFLOW_CLI_LOCALE=pt-BRhappy-path: 1 credit, ~2.5 MB mp4,MEDIA_GENERATION_STATUS_SUCCESSFUL. Magic bytesftypisomconfirmed.VideoStatusreturn type (now wrapped inVideoResult). Filed as a separate follow-up; not in this PR.Test plan
All links resolved across 9 files.255 tracked files checked — no violations.