feat: sync feature-branch label

src/context/repoContext.ts

@@ -16,7 +16,7 @@ export interface LockedMergePr {
 
 interface RepoContextWithoutTeamContext<GroupNames extends string> {
   labels: Labels;
-  protectedLabelIds: LabelResponse['id'][];
+  protectedLabelIds: readonly LabelResponse['id'][];
 
   hasNeedsReview: (labels: LabelResponse[]) => boolean;
   hasRequestedReview: (labels: LabelResponse[]) => boolean;
@@ -76,6 +76,15 @@ async function initRepoContext<GroupNames extends string>(
     .filter(Boolean)
     .map((name) => labels[name].id);
 
+  const protectedLabelIds = [
+    ...requestedReviewLabelIds,
+    ...changesRequestedLabelIds,
+    ...approvedReviewLabelIds,
+  ];
+  if (labels['feature-branch']) {
+    protectedLabelIds.push(labels['feature-branch'].id);
+  }
+
   const labelIdToGroupName = new Map<LabelResponse['id'], GroupNames>();
   reviewGroupNames.forEach((key) => {
     const reviewGroupLabels = config.labels.review[key] as any;
@@ -147,11 +156,7 @@ async function initRepoContext<GroupNames extends string>(
 
   return Object.assign(repoContext, {
     labels,
-    protectedLabelIds: [
-      ...requestedReviewLabelIds,
-      ...changesRequestedLabelIds,
-      ...approvedReviewLabelIds,
-    ],
+    protectedLabelIds,
     hasNeedsReview,
     hasRequestedReview,
     hasChangesRequestedReview,

src/pr-handlers/actions/editOpenedPR.ts

@@ -148,6 +148,7 @@ export const editOpenedPR = async (
     await context.github.issues.update(context.issue(update));
   }
 
+  console.log(options);
   if (options && featureBranchLabel) {
     if (prHasFeatureBranchLabel && !options.featureBranch) {
       await context.github.issues.removeLabel(

src/pr-handlers/actions/utils/parseBody.test.ts

@@ -1,6 +1,7 @@
 import { parseBody } from './parseBody';
 import initialSimple from './mocks/body/initial-simple';
 import initialTable from './mocks/body/initial-table';
+import initialAfterEditSimple from './mocks/body/initialAfterEdit-simple';
 
 describe('simple', () => {
   it('should parse default description', () => {
@@ -35,3 +36,20 @@ describe('table', () => {
     });
   });
 });
+
+describe('table', () => {
+  it('should parse edited description', () => {
+    const defaultConfig = {
+      featureBranch: true,
+      deleteAfterMerge: true,
+    };
+
+    const parsed = parseBody(initialAfterEditSimple, defaultConfig);
+
+    expect(parsed).not.toBeFalsy();
+    expect(parsed && parsed.options).toEqual({
+      featureBranch: false,
+      deleteAfterMerge: true,
+    });
+  });
+});

src/pr-handlers/actions/utils/parseBody.ts

@@ -15,7 +15,7 @@ const parseOptions = (
       const match = regexp.exec(content);
       acc[name] = !match
         ? defaultConfig[name] || false
-        : match[1] === 'x' || match[2] === 'X';
+        : match[1] === 'x' || match[1] === 'X';
       return acc;
     },
     {} as any,

src/pr-handlers/actions/utils/prOptions.ts

@@ -4,7 +4,7 @@ export const options: Options[] = ['featureBranch', 'deleteAfterMerge'];
 export const optionsRegexps: { name: Options; regexp: RegExp }[] = options.map(
   (option) => ({
     name: option,
-    regexp: new RegExp(`\\[[ xX]]\\s*<!-- renovate-${option} -->`),
+    regexp: new RegExp(`\\[([ xX]?)]\\s*<!-- reviewflow-${option} -->`),
   }),
 );
 

src/teamconfigs/christophehurpeau.ts

@@ -62,6 +62,12 @@ const config: Config<'dev'> = {
         name: ':soon: automerge',
         color: '#64DD17',
       },
+
+      /* feature-branch */
+      'feature-branch': {
+        name: 'feature-branch',
+        color: '#7FCEFF',
+      },
     },
 
     review: {