Skip to content

Add AI Workflow guide - #11

Open
MihailTodorov wants to merge 1 commit into
masterfrom
add-ai-workflow
Open

Add AI Workflow guide#11
MihailTodorov wants to merge 1 commit into
masterfrom
add-ai-workflow

Conversation

@MihailTodorov

Copy link
Copy Markdown
Contributor

No description provided.

Comment on lines +177 to +179
The tools fall into four groups. You rarely call them yourself - the agent does,
while it works - but it is worth knowing what is available, because that tells you
what you can reasonably ask for.

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.

IMO this sentence needs a rewrite. It flows a bit weird, especially the "the agent does, while it works - but it is worth knowing..."

Comment on lines +30 to +32
:::caution[Gameface Player only]
The server refuses to launch or connect to anything that is not Gameface Player.
Every connection is verified over CDP - `navigator.userAgent` must contain

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.

I thought it can work directly with the game as well. If it really does not currently work with a game, we should put a sentence in this aside to state it and make it 100% clear

Comment on lines +273 to +275
## Checking it works

Ask your agent to launch the Player and screenshot your current view. If it comes

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.

What should they do if it doesn't work ? Should we add a "Troubleshooting" section after this one, for people that can't run it for whatever reason?

Comment on lines +97 to +100
The six `09_*` files are the Gameface UI component library, split by category, and
they are the ones that pay off fastest in day-to-day work: an agent that knows
`Slider`, `Navigation` and `ScrollView` already exist will use them instead of
building its own out of divs.

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.

I think this will sit better in an aside (note or caution). And also we should emphasize that we are talking strictly in a "Gameface UI" project.. not in any other. Just to make it 100% clear for people that are reading but aren't as acquainted with our tools and gf ui.

permanently in its unchecked state.

:::caution[The ruleset is a snapshot, not a contract]
It reflects what feature detection found on the versions it was run against, and

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.

The heading reads a bit weird imo

<Link href="https://frontend-tools.coherent-labs.com/eslint-plugin-gameface/getting-started/">eslint-plugin-gameface</Link>
flags unsupported patterns and data-binding mistakes across CSS, SCSS, HTML,
JSX/TSX and JavaScript. It ships pre-wired in the Gameface UI boilerplate and in
every framework template.

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.

Suggested change
every framework template.
every <Link href="/planning-and-setup/alternative-frameworks/overview/">framework template.</Link>

Comment on lines +77 to +78
The [Conductor skill](/building-with-ai/the-conductor-skill/) runs this battery
automatically as its validation stage. Without the skill, ask for it explicitly.

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.

Suggested change
The [Conductor skill](/building-with-ai/the-conductor-skill/) runs this battery
automatically as its validation stage. Without the skill, ask for it explicitly.
The [Conductor skill](/building-with-ai/the-conductor-skill/) runs this
automatically as its validation stage. Without the skill, ask for it explicitly.

battery? :D

Comment on lines +78 to +79
Both were decided before a line was written. Discovering either one in week three
is a rewrite.

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.

The "week" jargon sounds a bit off here ;d
Can we emphasize on project longevity in a different way ?


Written before the first chart, not measured afterwards:

> Target ≤ 200 marks; warn in dev above 500. Exactly one tween signal write per

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.

What is a mark?

> measured off the DOM. The chart emits its own geometry, so the length is
> *derived*.

An earlier decision, recorded with its reasoning, was revisited months later and

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.

Wasn't it made in a week or so why "months later" ?


## Layer 1 - Lint the source

The cheapest check, and it needs no agent involvement.

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.

In fact if you have eslint plugin setup in your project it is possible Claude or other AI to try to run and check for mistakes. Maybe, describe this possibility here.

* **Visual coherence** - does this belong to the same interface as the screen next
to it? See [Design first](/building-with-ai/design-first/).
* **Hierarchy** - does the most important thing read as the most important thing?
* **Feel** - transitions, timing, feedback on press. An agent will add a 300ms

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.

"An agent will add a 300ms ease to everything and call it polish." Will this happen every time? I don't understand the context of this sentence.


This is the layer people skip, and it is the one that caught the worst bug we hit.

While building the Chart component, an overlay element that should have covered the

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.

Who has built the Chart component? Why we mention Chart component out of nowhere and out of context :D

Comment on lines +18 to +23
:::caution[An internal experiment, and still in development]
The Chart family is being developed for the Gameface UI library and is not
released. This article is about the working method, not an announcement. See the
<Link href="https://gameface-ui.coherent-labs.com/components/">component library</Link>
for what is actually available today.
:::

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.

Are we sure we want to mention that it is not released?

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.

3 participants