Frontmatter
| id | 7287 |
| title | Convert state/FeedbackLoop.mjs Test from Siesta to Playwright |
| state | Closed |
| labels | enhancementhelp wantedgood first issuehacktoberfest |
| assignees | nabeel001 |
| createdAt | Sep 27, 2025, 3:59 PM |
| updatedAt | Oct 10, 2025, 1:07 PM |
| githubUrl | https://github.com/neomjs/neo/issues/7287 |
| author | tobiu |
| commentsCount | 7 |
| parentIssue | 7262 |
| subIssues | [] |
| subIssuesCompleted | 0 |
| subIssuesTotal | 0 |
| blockedBy | [] |
| blocking | [] |
| closedAt | Oct 10, 2025, 1:07 PM |
Convert state/FeedbackLoop.mjs Test from Siesta to Playwright

hey @tobiu, I'd like to work on this, please assign this to me.

Hi and thanks for your interest. For the testing tickets, I strongly recommend trying out the “ai native” workflow.
I would read the hacktoberfest intro first: https://github.com/neomjs/neo/issues/7296
Then the following 2 guides: https://github.com/neomjs/neo/blob/dev/.github/WORKING_WITH_AGENTS.md https://github.com/neomjs/neo/blob/dev/.github/AI_QUICK_START.md
E.g. gemini cli should be capable to complete it on its own.

Hi @tobiu In case @shubham220420 is unable to resolve this issue, Kindly assign this to me.

Hey @nabeel001 Its not that I am not able to solve it but currently I am a bit busy in my exams, so you can take over this problem :)

Sure @shubham220420 No worries, All the best with your exams. I'll take care of this issue.
@tobiu kindly assign this to me

Hi guys, sure, I can re-assign it. This was interesting on a meta-level: I just updated our contributing md file, to add:
To keep the project moving, if a ticket is assigned and we do not receive any feedback from the assignee (e.g. via a comment) within 7 days, the ticket may get re-assigned to make it available for other contributors.
https://github.com/neomjs/neo/issues/7426 (also updated it, since it was extremely outdated).
Good luck with your exams from me too!
And no worries: even when this epic is finished, there are a lot more files which could use unit testing (Gemini can identify them, and create new tickets). Plus, there is also component based testing, which would be a full new epic.

Hi @tobiu Kindly review and merge the PR.
This task is to migrate the unit test for
state/FeedbackLoop.mjsfrom the Siesta test harness to the Playwright test runner.Acceptance Criteria
test/playwright/unit/state/FeedbackLoop.spec.mjs.test/siesta/tests/state/FeedbackLoop.mjs) to the Playwright/Jestexpectsyntax.npm test.