v0.1.0-alpha: Feedback

[azinit]

@Krakazybik I suggest to collect here known issues after using 0.1.0 prototype versions (for quick fix)

[azinit]

Problem: Not working linting of private segments
Rules: public-api-boundaries
Version: 0.0.0-concept.2
Reference: https://github.com/ani-team/github-client/blob/workshop/feature-sliced-next/src/widgets/header/index.tsx
Issue: #53

[Krakazybik]

проблема связана с доступом по пути features/repo-search/ui.tsx

[Krakazybik]

Fixed in #54

[azinit]

Problem: False trigger on valid imports
Rules: public-api
Version: 0.1.0-alpha
Issue: #60

[Krakazybik]

Try to fix in #61

[azinit]

Problem: Incorrect processing of aliased-imports
Rules: import-order
Version: 0.1.0-alpha.1
Reference: https://github.com/tednaaa/paint
Issue: #58

Screenshots

[azinit]

Other examples: https://github.com/illright/draw-ill-help

[Krakazybik]

partial fix in #64

[azinit]

А поч "частично"?)
Разве поддержка альясов не налаживает корректный порядок? Или не тестил еще?

[Krakazybik]

Ну частично, из-за того, что мы решили сделать поддержку только для корневых альясов, аля "@" "@root" "~". А альясы вроде "@lib => /src/shared/lib" решили не реализовывать, т.к. это потребует совсем другую реализацию.

[azinit]

А, ну это то да)

[azinit]

Problem: Falsy triggered on imports of local modules
Rules: public-api
Version: 0.1.0-alpha.1
Reference: https://github.com/tednaaa/paint

[azinit]

[azinit]

В целом то он верно с какой-то стороны отлавливает

Но кажется реэкспортить внутренности на уровне сегментов - вполне легально
(в идеале такой потом отдельным правилом линтить)

[Krakazybik]

Хм.. Как мне кажется не стоит делать такие послабления на уровне basic eslint-config'a, т.к. eslint - это всё таки набор бэст практикс.

Это по сути отключение обязательности public-api (т.е. отключение правила), даже из скринов мы можем видет что изменнение названия tool.model - скорее всего потянет за собой изменеение во всех интруменах. Нарушает ли это требования (а также практик SOLID/etc) устойчивости к изменениям PublicAPI? Мне кажется, отчасти, да. Является ли это бэст практикс? Я хочу от тебя услышать =)

Методолгия предоставляет множество возможностей и старается по минимумы ограничивать и многие вещи выглядят как "так лучше не делать, но делать можно" (и я не уверен, что все эйдж кейсы мы сможем завернуть в конфиг), но, как мне кажется, это не про eslint-config , тем более ты сам знаешь о его возможностях =)

Врядли мы сможем подстроиться под всех на данном этапе и с текущими возможностями конфига.
И да, возможно в дальнейшем, на уровне плагина, стоит ввести возможность кастомизации доступа для Public.

[azinit]

т.к. eslint - это всё таки набор бэст практикс.

"так лучше не делать, но делать можно" (и я не уверен, что все эйдж кейсы мы сможем завернуть в конфиг), но, как мне кажется, это не про eslint-config , тем более ты сам знаешь о его возможностях =)

Да, тут история скорее про то, что public-api, как и любая концепция FSD, имеет несколько степеней применимости согласно строгости/профитности

1. public-api#slices - реэкспорты на уровне слайсов обязательны
2. public-api#segments - + реэкспорты на уровне сегментов обязательны
3. public-api#absolute - + реэкспорты обязательны везде

Тут речь как раз про то, что хоть eslint-config и может линтить сразу на уровне 3, но люди еще адаптируются и пытаются принять методологию только на уровне 1

А если и eslint их настолько блочит, то могут быть проблемы (хотя про бест практис с твоей стороны - тоже резонно в каком-то виде)

[azinit]

Врядли мы сможем подстроиться под всех на данном этапе и с текущими возможностями конфига.
И да, возможно в дальнейшем, на уровне плагина, стоит ввести возможность кастомизации доступа для Public.

Согласен, и тут скорее "замечание на будущее" нам

Т.к. сплитить щас эти уровни кажется рили будет проблемным, но важным позднее

[azinit]

Problem: Shared should be able to import ANY other shared-segments
Rules: layers-slices
Version: 0.1.0-alpha.1
Reference: https://github.com/tednaaa/paint

[azinit]

Позже наверное еще добавим правило segments-boundaries, чтобы соблюдался порядок:

[ANY_CUSTOM_SEGMENT] => ui => model => api => lib => config

Для shared тоже будет работать, но ща надо разрешить shared импортить любой ресурс из shared

[Krakazybik]

так и не добавили shared сегменты

[azinit]

так и не добавили shared сегменты

@Krakazybik Ну вот надо бы добавить этот слой как исключение)

[Krakazybik]

Fixed in #65

[azinit]

Problem: Is no supported for .svelte files
Rules: import-order
Version: 0.1.0-alpha.1
Reference: https://github.com/illright/draw-ill-help

[azinit]

И как полагаю - для .vue скорее всего тоже. Но точнее - позже скажу

[azinit]

Хотя для каких то работает 🤔