[detectadd] packages_dir 가 루트도 볼 수 있게 해주세요!

[yceffort-naver]

작업 내용

https://github.com/NaverPayDev/hidash/blob/2f9148d4bc5b7c5046cf3d6e9686ef6305963730/.github/workflows/detect-changed-packages.yml#L38

NaverPayDev/hidash#10

[github-actions]

@yceffort-naver 님, 이슈에 label 을 작성해 주세요.

[npayolaf]

1. excludes 패턴 추가하기 (경로로 분리)
excludes: "packages/foo, .github"

 // 특정 패턴에 따른 제외 처리
const �제외조건에해당하는파일인가 = excludes.some(exclude => {
    return filename === exclude || filename.startsWith(`${exclude}`);
});

2. packages_dir: "." 루트도 받을 수 있도록 고려하기

• 지금은 package/ 과 같이 고정된 경로만 받을 수 있음
• react/foo/package.json 과 같이 폴더 구조가 깊어질 수 있음을 고려하여 코드 변경 필요 (재귀 형식으로 변경된 파일에서 가까운 package.json 찾도록

function findNearestPackageJson(filePath) {
  let 현재경로 = path.dirname(filePath);
    
  while (현재경로 !== path.parse(현재경로).root) {
    const packageJsonPath = path.join(currentDir, 'package.json');
                    
    if (fs.existsSync(packageJsonPath)) {
      return packageJsonPath;
    }
                    
    현재경로 = path.dirname(현재경로);
  }

  return undefined; 
}

3. 유효한 폴더인지 조건 변경

const 루트를포함하는가 = packagesDir.includes(".") === true
const 패키지폴더 = packagesDir.filter((packagename) => packagename !== '.')

// 루트를 포함한다면 패스
const isTargetDirectories = 루트를포함하는가 || 패키지폴더.some((packageDir) => filename.includes(`${packageDir}/`))

**4. 루트를 포함한다면 package.json 만 리턴하도록 **

const 패키지경로들 = 루트를포함하는가 === true ? ['package.json'] : 2번재귀로직

5. package.json 에서 name 추출하는 로직 간소화해도될듯 (못찾으면 일단 에러 터트리자)

const packagesNames = 패키지경로들.map( (packageJsonPath) => {
        const packageJsonData = fs.readFileSync(packageJsonPath, 'utf-8');
        const packageJson = JSON.parse(packageJsonData);
        
        return packageJson.name
    })

[npayolaf]

canary 쪽도 비슷한 이슈가 있을듯 .. 일단 하나하나 ..

[yceffort]

wow..