feat: add process.getSystemVersion() (#16599)

atom/common/api/atom_bindings.cc

@@ -73,6 +73,8 @@ void AtomBindings::BindProcess(v8::Isolate* isolate,
   process->SetMethod("getHeapStatistics", &GetHeapStatistics);
   process->SetMethod("getProcessMemoryInfo", &GetProcessMemoryInfo);
   process->SetMethod("getSystemMemoryInfo", &GetSystemMemoryInfo);
+  process->SetMethod("getSystemVersion",
+                     &base::SysInfo::OperatingSystemVersion);
   process->SetMethod("getIOCounters", &GetIOCounters);
   process->SetMethod("getCPUUsage", base::Bind(&AtomBindings::GetCPUUsage,
                                                base::Unretained(metrics)));

docs/api/process.md

@@ -17,6 +17,7 @@ In sandboxed renderers the `process` object contains only a subset of the APIs:
 - `getHeapStatistics()`
 - `getProcessMemoryInfo()`
 - `getSystemMemoryInfo()`
+- `getSystemVersion()`
 - `getCPUUsage()`
 - `getIOCounters()`
 - `argv`
@@ -206,6 +207,17 @@ Returns `Object`:
 Returns an object giving memory usage statistics about the entire system. Note
 that all statistics are reported in Kilobytes.
 
+### `process.getSystemVersion()`
+
+Returns `String` - The version of the host operating system.
+
+Examples:
+- macOS: `10.13.6`
+- Windows: `10.0.17763`
+- Linux: `4.15.0-45-generic`
+
+**Note:** It returns the actual operating system version instead of kernel version on macOS unlike `os.release()`.
+
 ### `process.takeHeapSnapshot(filePath)`
 
 * `filePath` String - Path to the output file.

spec/api-process-spec.js

@@ -59,6 +59,12 @@ describe('process module', () => {
     })
   })
 
+  describe('process.getSystemVersion()', () => {
+    it('returns a string', () => {
+      expect(process.getSystemVersion()).to.be.a('string')
+    })
+  })
+
   describe('process.getHeapStatistics()', () => {
     it('returns heap statistics object', () => {
       const heapStats = process.getHeapStatistics()