Address nits

Author: ravibrock

README.md

@@ -67,8 +67,8 @@ GitHub Action for automatically syncing LeetCode submissions to a GitHub reposit
 - `leetcode-session` _(required)_: The LeetCode session value for retrieving submissions from LeetCode
 - `filter-duplicate-secs`: Number of seconds after an accepted solution to ignore other accepted solutions for the same problem, default: 86400 (1 day)
 - `destination-folder` _(optional)_: The folder in your repo to save the submissions to (necessary for shared repos), default: _none_
-- `verbose` _(optional)_: Requires an additional API call but adds submission percentiles and question numbers to the repo, default: true
-- `commit-header` _(optional)_: How the automated commits should be prefixed, default: '[LeetCode Sync]'
+- `verbose` _(optional)_: Adds submission percentiles and question numbers to the repo (requires an additional API call), default: true
+- `commit-header` _(optional)_: How the automated commits should be prefixed, default: 'Sync LeetCode submission'
 
 ## Shared Repos
 

action.yml

@@ -22,13 +22,13 @@ inputs:
     required: false
     default: null
   verbose:
-    description: 'Requires an additional API call but adds submission percentiles and question numbers to the repo'
+    description: 'Adds submission percentiles and question numbers to the repo (requires an additional API call)'
     required: false
     default: true
   commit-header:
     description: 'How the automated commits should be prefixed'
     required: false
-    default: '[LeetCode Sync]'
+    default: 'Sync LeetCode submission'
 runs:
   using: 'node16'
   main: 'index.js'

src/action.js

@@ -1,7 +1,7 @@
 const axios = require('axios');
 const { Octokit } = require('@octokit/rest');
 
-const COMMIT_MESSAGE = '[LeetCode Sync]';
+const COMMIT_MESSAGE = 'Sync LeetCode submission';
 const LANG_TO_EXTENSION = {
   'bash': 'sh',
   'c': 'c',
@@ -34,6 +34,9 @@ function log(message) {
 }
 
 function pad(n) {
+    if (n.length > 4) {
+        return n;
+    }
     var s = '000' + n;
     return s.substring(s.length-4);
 }
@@ -74,7 +77,7 @@ async function getInfo(submission, session, csrf) {
       const questionId = pad(response.data.data.submissionDetails.question.questionId.toString());
 
       log(`Got info for submission #${submission.id}`);
-      return { runtimeperc: runtimePercentile, memoryperc: memoryPercentile, qid: questionId };
+      return { runtimePerc: runtimePercentile, memoryPerc: memoryPercentile, qid: questionId };
     } catch (exception) {
       if (retryCount >= maxRetries) {
         throw exception;
@@ -114,8 +117,8 @@ async function commit(params) {
   const prefix = !!destinationFolder ? `${destinationFolder}/` : '';
   const commitName = !!commitHeader ? commitHeader : COMMIT_MESSAGE;
 
-  if ('runtimeperc' in submission) {
-    message = `${commitName} Runtime - ${submission.runtime} (${submission.runtimeperc}), Memory - ${submission.memory} (${submission.memoryperc})`;
+  if ('runtimePerc' in submission) {
+    message = `${commitName} Runtime - ${submission.runtime} (${submission.runtimePerc}), Memory - ${submission.memory} (${submission.memoryPerc})`;
     qid = `${submission.qid}-`;
   } else {
     message = `${commitName} Runtime - ${submission.runtime}, Memory - ${submission.memory}`;

src/test_config.js

@@ -11,5 +11,5 @@ module.exports = {
   FILTER_DUPLICATE_SECS: 86400,
   DESTINATION_FOLDER: null,
   VERBOSE: true,
-  COMMIT_HEADER: '[LeetCode Sync]'
+  COMMIT_HEADER: 'Sync LeetCode submission'
 }