fix: apply Copilot review suggestions

- README.md: clarify MATLAB/Verilog only support flat numeric arrays
- concore_base.hpp: add #include <stdexcept> and <cctype> for portability
- concore_base.hpp: fix keyword boundary checks to also exclude underscore
- concore.hpp: add Doxygen docs for parse_literal and flatten_numeric

Author: GaneshPatil7517

README.md

@@ -23,7 +23,7 @@ _concore_ enables composing studies from programs developed in different languag
 
 ## Wire Format
 
-Concore payloads follow Python literal syntax compatible with `ast.literal_eval()`. All language implementations (Python, C++, Java, MATLAB) parse this shared format. Supported value types include:
+Concore payloads follow Python literal syntax compatible with `ast.literal_eval()`. The Python, C++, and Java implementations parse this shared format; the MATLAB and Verilog implementations currently support only flat numeric arrays derived from it. Supported value types include:
 
 * **Numbers** — integers and floats, including scientific notation (e.g., `1e3`, `-2.5`)
 * **Booleans** — `True` / `False` (converted to `1.0` / `0.0` in numeric contexts)

concore.hpp

@@ -337,10 +337,20 @@ class Concore{
         return concore_base::parselist_double(f);
     }
 
+    /**
+     * @brief Parses a literal string into a ConcoreValue representation.
+     * @param f The input string to parse.
+     * @return A ConcoreValue obtained by parsing the input string.
+     */
     concore_base::ConcoreValue parse_literal(string f){
         return concore_base::parse_literal(f);
     }
 
+    /**
+     * @brief Flattens a ConcoreValue into a vector of numeric (double) values.
+     * @param v The ConcoreValue to flatten.
+     * @return A vector of double values obtained by flattening the input.
+     */
     vector<double> flatten_numeric(const concore_base::ConcoreValue& v){
         return concore_base::flatten_numeric(v);
     }

concore_base.hpp

@@ -14,6 +14,8 @@
 #include <regex>
 #include <algorithm>
 #include <cstring>
+#include <stdexcept>
+#include <cctype>
 
 namespace concore_base {
 
@@ -188,17 +190,20 @@ inline ConcoreValue parse_literal_value(const std::string& s, size_t& pos) {
         return parse_literal_string(s, pos);
 
     if (s.compare(pos, 4, "True") == 0 &&
-        (pos + 4 >= s.size() || !std::isalnum(static_cast<unsigned char>(s[pos + 4])))) {
+        (pos + 4 >= s.size() ||
+         (!std::isalnum(static_cast<unsigned char>(s[pos + 4])) && s[pos + 4] != '_'))) {
         pos += 4;
         return ConcoreValue::make_bool(true);
     }
     if (s.compare(pos, 5, "False") == 0 &&
-        (pos + 5 >= s.size() || !std::isalnum(static_cast<unsigned char>(s[pos + 5])))) {
+        (pos + 5 >= s.size() ||
+         (!std::isalnum(static_cast<unsigned char>(s[pos + 5])) && s[pos + 5] != '_'))) {
         pos += 5;
         return ConcoreValue::make_bool(false);
     }
     if (s.compare(pos, 4, "None") == 0 &&
-        (pos + 4 >= s.size() || !std::isalnum(static_cast<unsigned char>(s[pos + 4])))) {
+        (pos + 4 >= s.size() ||
+         (!std::isalnum(static_cast<unsigned char>(s[pos + 4])) && s[pos + 4] != '_'))) {
         pos += 4;
         return ConcoreValue::make_string("None");
     }