Conversation
Refs #684 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Refs #684 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
…e build compiles Refs #684 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Refs #684 Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Contributor
Changepacks@devup-ui/wasm@1.0.82 → 1.0.83 - bindings/devup-ui-wasm/package.jsonPatch
@devup-ui/bun-plugin@1.0.21 → 1.0.22 - packages/bun-plugin/package.jsonPatch
@devup-ui/components@0.1.59 → 0.1.60 - packages/components/package.jsonPatch
@devup-ui/eslint-plugin@1.0.21 → 1.0.22 - packages/eslint-plugin/package.jsonPatch
@devup-ui/next-plugin@1.0.89 → 1.0.90 - packages/next-plugin/package.jsonPatch
@devup-ui/plugin-utils@1.0.16 → 1.0.17 - packages/plugin-utils/package.jsonPatch
@devup-ui/rsbuild-plugin@1.0.66 → 1.0.67 - packages/rsbuild-plugin/package.jsonPatch
@devup-ui/vite-plugin@1.0.72 → 1.0.73 - packages/vite-plugin/package.jsonPatch
@devup-ui/webpack-plugin@1.0.70 → 1.0.71 - packages/webpack-plugin/package.jsonPatch
|
Codecov Report✅ All modified and coverable lines are covered by tests.
🚀 New features to boost your workflow:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refs #684 (TOOL-03, TOOL-04, TOOL-05 / KNOWN-15, W4에서 보고한 한계)
#733(
fix/eslint-autofix-scope) 위에 쌓인 PR입니다. base는 #733 브랜치이고, #733이 먼저 병합되어야 합니다(병합되면 base를main으로 바꿉니다).요약
css-utils-literal-only가 빌드가 기본으로 컴파일하는 import alias(@emotion/react,@emotion/styled,styled-components,@vanilla-extract/css)와 그 스타일 위치를 그대로 따릅니다. 위치:css/keyframes/createGlobalStyle호출,styled.div({...}),styled('div')({...}),styled('div', {...}),styled(Base)({...}, {...}), Emotion<Global styles>, VEstyle/globalStyle(일반 모듈에서)..attrs()/.withConfig()인자와 CSS 텍스트는 스타일이 아닙니다.const s = { w: [1, 1] }; css(s)처럼 빌드가 인라인하는 상수 객체/배열을, export되지 않고 바뀌지 않고 스타일로만 읽힐 때 선언 위치에서 검사/수정합니다.@devup-ui/no-runtime-read(recommended 포함):css/keyframes/styled/Box를 값으로 읽는 코드를 빌드 에러와 같은 의미의 메시지로 미리 보고합니다(KNOWN-15, 맨styled.div).css-utils-literal-only: Emotion/styled-components 별칭 호출과 Emotion<Global styles>의 동적 값을 보고하고,.css.ts/.css.js(빌드가 통째로 실행)에서는 실행 환경에 없는 전역(window,document,process,setTimeout,fetch,URL등, 빌드에서 직접 확인한 목록)의 읽기를 보고합니다(typeof가드는 허용).동작
.css.ts를 VE로 실행하는 파일에서는 배열이 반응형이 아니므로(대체 값) 모든 스타일 규칙이 꺼집니다. 다른 패키지(@emotion/css등, 기본 alias에 없음)는 건드리지 않습니다.no-runtime-read는 빌드의CompiledReads와 같은 기준입니다: 호출/태그/요소/as={Text}/styled(Box)(...)/타입(typeof css)/최상위 별칭(const a = css)만 허용하고, 함수 안의 별칭 초기화,css.foo,[Box],createElement(Box),<Box.Foo />,styled['div'], 끝나지 않은styled(Box)체인 등은 보고합니다..css.ts예제라서 해당 페이지에서만css-utils-literal-only를 끄도록eslint.config.mjs에 override를 넣었습니다.새로 생기는 오류
@devup-ui/no-runtime-read(error, recommended),.css.ts에서 없는 전역 읽기(css-utils-literal-only의missingGlobal),<Global styles>/별칭 호출의 동적 값.남는 한계
importAliases옵션)와@emotion/css처럼 기본에 없는 패키지는 lint가 알 수 없어 다루지 않습니다(빌드 설정을 읽지 않음).cssprop이 아직 없어 다루지 않았습니다.styleOrder는 상수 객체를 따라가지 않고 호출/요소에 직접 쓴 것만 봅니다. 네임스페이스 import(Devup.css) 값 읽기는 빌드도 보고하지 않으므로no-runtime-read가 보지 않습니다.const inner = { w: [2, 2] }; css(inner)는 빌드가 에러 없이inner(객체 자체)를 결과로 남깁니다. 로컬 상수 객체를 읽지 못하는 조용한 오동작이라 별도 수정이 필요합니다.검증
bun test5816 pass / 0 fail, 커버리지 100%.bun lint오류 0(기존 경고 2개). Rust 변경 없음.let/다른 읽기 → 건드리지 않음),.css.ts무시,no-runtime-read허용/보고 80여 건,.css.ts전역 읽기,<Global styles>동적 값.probe.tsvsESLint):export const runtime = css→ 빌드2:24 css is read at runtime, lint 같은 위치export const a = styled.div.attrs({})→ 빌드/lintstyled보고,styled(Box)→styled와Box둘 다 보고const z = css; z({p:1})허용, 함수 안const q = css→css보고(빌드와 동일)<Global styles={{ body: { color: window.name } }} />→ 빌드/lint 보고.css.ts의style({ color: window.name })→ 빌드ReferenceError: window is not defined, lintwindow보고styled('div', { w: [1, 1, null] })등 → 빌드가 읽는 스타일이므로[1, null]/[1]로 수정,.attrs({ w: [1, 1] })와<Global data-x={{ w: [1, 1] }} />는 변경 없음