Feat/theme generator improvement #29

aliceinapple · 2024-06-13T22:26:56Z

Resilves #28

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

Нейминги для радиусов и высоты линии сделала по аналогии с размерами шрифтов
Нейминги для теней, шрифтов и весов шрифтов оставила как есть (для весов и самих шрифтов нейминги вроде как логичные, на счет теней вопрос, сейчас у них нейминги генерируются в соответствии с цветом)

Структура для кнопок и инпутов
Посмотрела несколько вариантов разных макетов и сделала генерацию структуры в цветах для кнопок и инпутов в соответствии с графой "Архив" в ридми, это будет самый удобный вариант для генерации

…refix

Nelfimov

Отличная работа!

Давай попробуем перевести все в ESM, а так же покрыть тестами. Я вижу так:

либо покрываем тестом каждую функцию класс (юнит тесты)
либо в качестве теста прогоняем генератором какой нибудь дизайн старого проекта (напр. drum-in), делаем снэпшот (можно в виде строки значений каждого объекта) и сравниваем с ним (интеграционный). Примеры можешь посмотреть в гиперионе генераторе или тулзах.

В идеале сделать оба варианта, но если уйдет много времени - давай что-то одно сделаем.

.yarnrc.yml

package.json

aliceinapple · 2024-06-16T22:05:24Z

@Nelfimov

После перехода на ESM, когда запускаю скрипт - получаю ошибку:

TypeError [ERR_UNKNOWN_FILE_EXTENSION]: Unknown file extension ".ts" for /Users/macbookair15/WebstormProjects/figma/theme/theme-cli/src/index.ts
    at Object.getFileProtocolModuleFormat [as file:] (node:internal/modules/esm/get_format:160:9)
    at defaultGetFormat (node:internal/modules/esm/get_format:203:36)
    at defaultLoad (node:internal/modules/esm/load:143:22)
    at async nextLoad (node:internal/modules/esm/hooks:750:22)
    at async nextLoad (node:internal/modules/esm/hooks:750:22)
    at async nextLoad (node:internal/modules/esm/hooks:750:22)
    at async Hooks.load (node:internal/modules/esm/hooks:383:20)
    at async handleMessage (node:internal/modules/esm/worker:199:18) {
  code: 'ERR_UNKNOWN_FILE_EXTENSION'
}

Пробовала также разные другие скрипты, например такой node --loader ts-node/esm src/index.ts, получаю другую ошибку:

node:internal/process/esm_loader:34
      internalBinding('errors').triggerUncaughtException(
                                ^
Error: Cannot find package 'commander' imported from /Users/macbookair15/WebstormProjects/figma/theme/theme-cli/src/index.ts
    at packageResolve (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/dist-raw/node-internal-modules-esm-resolve.js:757:9)
    at moduleResolve (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/dist-raw/node-internal-modules-esm-resolve.js:798:18)
    at Object.defaultResolve (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/dist-raw/node-internal-modules-esm-resolve.js:912:11)
    at /Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:218:35
    at entrypointFallback (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:168:34)
    at /Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:217:14
    at addShortCircuitFlag (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:409:21)
    at resolve (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:197:12)
    at nextResolve (node:internal/modules/esm/hooks:750:28)
    at Hooks.resolve (node:internal/modules/esm/hooks:238:30)

Node.js v21.7.3

TorinAsakura · 2024-06-16T22:12:04Z

@aliceinapple Ноду откати на 19

aliceinapple · 2024-06-17T07:50:39Z

@TorinAsakura @Nelfimov

@aliceinapple Ноду откати на 19

Все равно остается проблема с импортом пакетов (пробовала еще 18 ноду)

/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/dist-raw/node-internal-modules-esm-resolve.js:757
  throw new ERR_MODULE_NOT_FOUND(packageName, fileURLToPath(base));
        ^
CustomError: Cannot find package 'commander' imported from /Users/macbookair15/WebstormProjects/figma/theme/theme-cli/src/index.ts
    at packageResolve (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/dist-raw/node-internal-modules-esm-resolve.js:757:9)
    at moduleResolve (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/dist-raw/node-internal-modules-esm-resolve.js:798:18)
    at Object.defaultResolve (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/dist-raw/node-internal-modules-esm-resolve.js:912:11)
    at /Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:218:35
    at entrypointFallback (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:168:34)
    at /Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:217:14
    at addShortCircuitFlag (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:409:21)
    at resolve (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/esm.ts:197:12)
    at resolve (/Users/macbookair15/WebstormProjects/figma/.yarn/__virtual__/ts-node-virtual-1eb244db5c/0/cache/ts-node-npm-10.9.2-3f3890b9ac-a91a15b3c9.zip/node_modules/ts-node/src/child/child-loader.ts:15:39)
    at nextResolve (node:internal/modules/esm/hooks:654:28)
macbookair15@MacBook-Air-Macbook theme-cli % node -v
v19.9.0

Nelfimov · 2024-06-17T09:34:03Z

@aliceinapple добавил новый скрипт на swc. Так же пришлось бампнуть commander. Проверено на 22 ноде. Для 19 придется менять --import swc-node/register/esm-register на --loader swc-node/register/esm

aliceinapple added 12 commits June 7, 2024 18:16

chore: update yarn

3fbeef1

feat: add the ability to add pages to ignore

08da718

feat: add the ability to generate theme from selected pages

40ca157

chore: upgrade dependencies

20c19ae

feat: strategy for naming radii

6aa38b0

fix: check errors

0fc383a

feat: strategy for naming line heights

516b0d2

refactor: theme-cli

3afa7f2

feat: add the ability to generate theme from elements with specific p…

61299b9

…refix

refactor: colors generator

209feef

feat: add function to get input states

033f76b

refactor: colors generator

354b655

aliceinapple added the feature label Jun 13, 2024

aliceinapple requested a review from Nelfimov June 13, 2024 22:26

aliceinapple self-assigned this Jun 13, 2024

aliceinapple linked an issue Jun 13, 2024 that may be closed by this pull request

Доработка генератора #28

Closed

3 tasks

Nelfimov requested changes Jun 14, 2024

View reviewed changes

.yarnrc.yml Show resolved Hide resolved

package.json Outdated Show resolved Hide resolved

aliceinapple added 13 commits June 14, 2024 10:36

chore: upgrade typescript

fc6c696

chore: transition to ESM

563bd94

chore: update jest

c59504b

feat: add unit test to font weights generator

b653b08

feat: add unit test to fonts generator

d997460

feat: add unit test to font sizes generator

4188219

feat: add unit test to line heights generator

844654d

feat: add unit test to radii generator

f0275a5

feat: add unit test to shadows generator

f42fe6f

refactor: rename tests

63b85ae

chore: upgrade dependencies

fc2f1da

feat: add test to colors generator

157e382

feat: add test to common generator

75ecf30

aliceinapple added 2 commits June 16, 2024 00:30

feat: add test to theme generator

b739081

feat: add test to run function

a9a879d

feat(theme-cli): bump versions, run via swc BREAKING CHANGE

d471b74

chore: change camelcase to lodash.camelcase

8f43285

aliceinapple requested a review from Nelfimov June 17, 2024 12:23

Nelfimov added 3 commits June 17, 2024 19:46

fix(tests): bump @atls/config-jest

24432fc

revert(common): camelcase

9f58bd7

feat(common): github actions

1685a21

Nelfimov approved these changes Jun 17, 2024

View reviewed changes

Nelfimov merged commit f28e447 into master Jun 17, 2024
5 checks passed

Nelfimov deleted the feat/theme-generator-improvement branch June 17, 2024 16:57

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/theme generator improvement #29

Feat/theme generator improvement #29

aliceinapple commented Jun 13, 2024 •

edited

Loading

Nelfimov left a comment

aliceinapple commented Jun 16, 2024

TorinAsakura commented Jun 16, 2024

aliceinapple commented Jun 17, 2024

Nelfimov commented Jun 17, 2024

Feat/theme generator improvement #29

Feat/theme generator improvement #29

Conversation

aliceinapple commented Jun 13, 2024 • edited Loading

Nelfimov left a comment

Choose a reason for hiding this comment

aliceinapple commented Jun 16, 2024

TorinAsakura commented Jun 16, 2024

aliceinapple commented Jun 17, 2024

Nelfimov commented Jun 17, 2024

aliceinapple commented Jun 13, 2024 •

edited

Loading