Repository navigation
docs(blueprints): screen specs, wireframes and component inventory - #146
Merged
Merged
Conversation
Refs #23. - docs/blueprints/screens.md: one section per screen ID of #22 (S01 to S10) with its purpose and flows, the data shown named as in docs/domain/model.md, the loading, empty, error and success states, the actions and where they lead, and the components it uses. - Component inventory: every reusable atom, molecule and organism of puls3_flutter/lib/src/ui, with the screens that use it. - docs/blueprints/wireframes: 32 PNGs, desktop (1440 px) and mobile (390 px) for every screen, plus loading, empty and error for S02 and S03. They are rendered by puls3_flutter/tool/wireframes (outside flutter test, so CI does not run them): built screens from the app with fake data, S05, S06 and S10 from the same components after F6 and F7. The test font draws text as blocks, so they read as low to mid fidelity wireframes. - flows.md links the specs and marks S09 as built (#25). The Figma link is left for the frontend owner to add. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
XxHugheadxX
approved these changes
Oct 9, 2026
XxHugheadxX
left a comment
Contributor
There was a problem hiding this comment.
Approved as a partial delivery of #23 (Refs #23).
screens.md: it covers S01–S10 with purpose, flows, data, the 4 states, actions and components.- Names:
- the data fields match
docs/domain/model.md, includingHire.runtimeStatusandHire.failureReason; - S05 and S06 use the ADR-0005 states and the approve, reject and refund actions from the updated flows (#140);
- a reject from
openreads "Cancelled".
- the data fields match
- Wireframes: every screen ID has
-1440and-390PNGs, and they come from the real components.
Before #23 can close:
- The Figma link is still an acceptance criterion of #23 and isn't here. Either add it, or agree on the issue to drop it in favour of the generated PNGs.
- Unreadable labels in the wireframes. The test font (Ahem) draws every glyph as a block, so no label can be read. For example, S06 doesn't show which button is Approve and which is Reject. Loading the app's real fonts in
tool/wireframesbefore rendering would make them readable, and it fits "low to mid fidelity". The goldens can do that withFontLoaderand the bundled font files. - Missing state wireframes. The other screens (S04–S10) document their loading, empty and error states in text only. That's allowed ("its own wireframe or a clear note"), but for S06 the states matter most, so one wireframe for its failed-run and approval-deadline states would help.
Review of #146. - The wireframe tool loads Roboto and the Material icons from the Flutter SDK (and the app's Doto), so every label reads instead of the test font's blocks. Puls3Fonts names an offline family (Roboto) for styles while Google Fonts are off; the app itself is unchanged, and tests keep the test font because they never load it. - S06 gets its failed-run and deadline-passed wireframes at both widths, linked from its states in screens.md. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Deploying puls3 with
|
| Latest commit: |
16472d4
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://728b0963.puls3-4lw.pages.dev |
| Branch Preview URL: | https://docs-23-screen-specs.puls3-4lw.pages.dev |
Contributor
Author
|
Thanks for the review! Done in 16472d4:
On Figma: I'll ask on #23 whether the generated PNGs can replace it, since they're regenerated from the app with one command and stay in sync with the code. |
This was referenced Oct 10, 2026
TOMOKI977
added a commit
that referenced
this pull request
Oct 11, 2026
…sion, my agents) and S11/S12 specs and wireframes (#36) (#161) flows.md: - F8 create and save a draft (saveDraft, StudioDraft.revision; InvalidManifest, DraftVersionConflict, DraftNotOwned, no answer); - F9 test run in the playground (testRun, TestRunResult.output and remainingDailyRuns; TestQuotaExceeded, AgentExecutionFailed, RuntimeUnavailable, InputTooLong); - F10 edit a deployed agent: version n+1 keeps the agent id and wallet, the builder signs the manifestHash/manifestVersion update, hires in flight keep version n (spike #33 section 4); - F11 my agents: drafts, deploying or failed (resume at retryFromStep), live (view, edit). F4 (deploy) is unchanged; the coverage and screen ID tables list the new flows and screens. screens.md: - S11-playground and S12-my-agents with data named after AgentManifest, StudioDraft, TestRunResult and DeploySession, the 4 states, actions and components; - S07/S08 list the flows they now serve; - the component inventory adds PlaygroundPanel, TestRunResultCard, MyAgentsList and AgentVersionRow. wireframes: S11 and S12 at 1440 and 390, in the #146 style. Co-authored-by: Julio Cesar Severiche Orellana <39506930+TOMOKI977@users.noreply.github.com>
6 tasks
4 of 5 tasks
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.
Refs #23
What
docs/blueprints/screens.md: one section per screen ID of docs: MVP user flows #22 (S01–S10), each with:docs/domain/model.md;puls3_flutter/lib/src/ui, with the screens that use it.docs/blueprints/wireframes/: 32 PNGs.puls3_flutter/tool/wireframes, which lives outsideflutter test, so CI does not run it: built screens use the app with fake data; S05, S06 and S10 are drawn from the same components after F6/F7.flows.mdlinks the specs and marks S09 as built (feat: wallet connection in Flutter app #25).Wireframes: the PNGs in
docs/blueprints/wireframes/. @TOMOKI977, let me know if you still need one.How to verify
flows.mdwithdocs/blueprints/wireframes/: every ID has-1440and-390.screens.mdand that the field names matchdocs/domain/model.md.cd puls3_flutter && flutter test tool/wireframes/wireframes_test.dart --update-goldensFrontend owner sign-off
As the frontend owner, I confirm these specs are enough to build every screen without design questions.