Skip to content

[Feat] Introduce linting in frontend project #234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 49 commits into
base: main
Choose a base branch
from
Open

[Feat] Introduce linting in frontend project #234

wants to merge 49 commits into from

Conversation

stefanoamorelli
Copy link
Contributor

@stefanoamorelli stefanoamorelli commented Jun 29, 2025 •
edited
Loading

Tip

Review commit-by-commit for easier review.

Solves: #133

We introduce ESlint for the frontend/, fix the existing issues, and add a Github workflow to run the linting and prevent merging of code that does not pass the linting.

@stefanoamorelli
refactor(eslint): implement custom ESLint rules for React and TypeScript
7ae502e
@stefanoamorelli
chore(package): add lint scripts and update package manager
34b795a
@stefanoamorelli
feat(ci): introduce frontend checks GitHub Action
911d4fe
@stefanoamorelli
feattypescript: add IPlan type import
2fe0a83
@stefanoamorelli
fix/runtime: ensure session status type safety
efb0382
@stefanoamorelli
feat(sidebar): improve session status type safety
20e4503
@stefanoamorelli
fix(signin): use submit type for sign in button
b55ccd7
@stefanoamorelli
feat(env): add environment variable loading
ff61be7
@stefanoamorelli
fix(markdown): remove node prop from code render
ee00213
@stefanoamorelli
refactor(button): simplify LearnPlanButton state management
2a415a3
@stefanoamorelli
feat(accessibility): improve keyboard navigation for file modal and c…
4f27609 
…ard components
@stefanoamorelli
fix(a11y): enable keyboard navigation for content header button
4f178d3
@stefanoamorelli
fix(404): escape apostrophe in page not found message
22c61bf
@stefanoamorelli
fix(Plans): add id and htmlFor to plan title input
05d50da
@stefanoamorelli
style(PlanList): remove unused props
4021a46
@stefanoamorelli
style(cleanup): remove redundant state initialization
cc4cc21
@stefanoamorelli
refactor(sidebar): remove unused imports and props
3c9ebe9
@stefanoamorelli
fix(statusicon): handle awaiting_input status
a870e05
@stefanoamorelli
refactor(provider): extract wrapper component
cdb5350
@stefanoamorelli
fix(search): add quotes to search term
d7cebe3
@stefanoamorelli
refactor(Layout): simplify UI component props
ed0ecd0
@stefanoamorelli
fix(modal): remove unused tabIndex attribute
a1c1e2d
@stefanoamorelli
fix(status): handle complete status
7cece54
@stefanoamorelli
fix(settings): remove unused imports
c03a7ed
@stefanoamorelli
refactor(statusicon): simplify icon logic
f71190f
@stefanoamorelli
fix(session management): prevent Promise resolution
6fd42bd
@stefanoamorelli
feat(modal): improve accessibility and keyboard navigation
6d88150
@stefanoamorelli
refactor(types): simplify datamodel imports
c36f89c
@stefanoamorelli
fix(chat): prevent crashes when processing messages
8f33817
@stefanoamorelli
fix(chat): ignore JSON parsing errors and improve fallback
81427d1
@stefanoamorelli
refactor(plan): simplify component logic and remove unused imports
d0d607c
@stefanoamorelli
refactor(chat): simplify component logic and remove unused state
6e04c83
@stefanoamorelli
refactor(chatinput): simplify component logic and remove unused imports
3488d0c
@stefanoamorelli
refactor(chat): simplify component logic and improve error handling
6240d9e
@stefanoamorelli
refactor(approval): simplify approval buttons logic
c186032
@stefanoamorelli
feat(accessibility): improve keyboard navigation in chat detail viewer
44fc927
@stefanoamorelli
fix(feedback): escape apostrophe in feedback form text
33f888d
@stefanoamorelli
feat(modal): improve keyboard navigation and accessibility
db1a95a
@stefanoamorelli
fix(api): handle missing or invalid plan data
264b83d
@stefanoamorelli
fix(modal): improve keyboard navigation
8700925
@stefanoamorelli
fix: handle error cases in chat view
e880e77
@stefanoamorelli
refactor(chatinput): simplify component props
bea7660
@stefanoamorelli
feat(detail viewer): add view mode property and control handover form
414ac81
@stefanoamorelli
fix(chat): remove onBlur event handler
905db80
@stefanoamorelli
refactor(RenderMessage): rename props for clarity
72d5d79
@stefanoamorelli
Merge branch 'main' into feat/linting-2
62bde8f
@stefanoamorelli stefanoamorelli mentioned this pull request Jun 29, 2025
Closed
@stefanoamorelli
Copy link
Contributor Author

@matheusmaldaner a new commit introduced 3 new errors hence CI was failing upon rebase, I've pushed another one to fix those 3 errors 👀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@stefanoamorelli